From: Peter Krempa <pkrempa@redhat.com> The 'error' label is not needed, we can directly return failure from the only error. Signed-off-by: Peter Krempa <pkrempa@redhat.com> --- tools/vsh-table.c | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/tools/vsh-table.c b/tools/vsh-table.c index da7dc84ee8..20d25eb227 100644 --- a/tools/vsh-table.c +++ b/tools/vsh-table.c @@ -92,7 +92,7 @@ vshTableRowNew(const char *arg, va_list ap) if (!arg) { virReportError(VIR_ERR_INTERNAL_ERROR, "%s", _("Table row cannot be empty")); - goto error; + return NULL; } row = g_new0(vshTableRow, 1); @@ -108,10 +108,6 @@ vshTableRowNew(const char *arg, va_list ap) } return row; - - error: - vshTableRowFree(row); - return NULL; } -- 2.53.0