From 0527cc793054e582ea8be8c0dc76df9e2ca0ab1b Mon Sep 17 00:00:00 2001 From: Garrett Wollman Date: Sun, 15 Jul 2012 14:54:47 -0400 Subject: [PATCH] util: in util_newTable, don't leak Table on error It is unlikely that this memory leak actually matters as the caller will normally just exit. This whole source file could stand to be reformatted. Change-Id: Ic5cb52a73ac09ae492c6e6ab606dfaf92a1ab8bf Reviewed-on: http://gerrit.openafs.org/7767 Tested-by: BuildBot Reviewed-by: Derrick Brashear --- src/util/tabular_output.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/util/tabular_output.c b/src/util/tabular_output.c index 5866fbef14..cfd43b26ed 100644 --- a/src/util/tabular_output.c +++ b/src/util/tabular_output.c @@ -369,6 +369,7 @@ util_newTable(int Type, int numColumns, char **ColumnHeaders, int *ColumnContent if (sortByColumn < 0 || sortByColumn > numColumns) { fprintf(stderr,"Invalid Table Sortkey: %d.\n", sortByColumn); errno=EINVAL; + free(Table); return NULL; } if (sortByColumn > 0 )