[PATCH v2] tools/vm: Fix possible resource leak in page_owner_sort.c

From: Saurabh Badhwar
Date: Mon Feb 20 2017 - 05:46:46 EST


The patch addresses explicit closing of file pointers on exit
and redirects the errors to the stderr
Signed-off-by: Saurabh Badhwar <sbsaurabhbadhwar9@xxxxxxxxx>
---
tools/vm/page_owner_sort.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/tools/vm/page_owner_sort.c b/tools/vm/page_owner_sort.c
index 8346261..b3c3efa 100644
--- a/tools/vm/page_owner_sort.c
+++ b/tools/vm/page_owner_sort.c
@@ -109,7 +109,7 @@ int main(int argc, char **argv)
list = malloc(max_size * sizeof(*list));
buf = malloc(BUF_SIZE);
if (!list || !buf) {
- printf("Out of memory\n");
+ perror("Unable to allocate memory: ");
fclose(fin);
fclose(fout);
exit(1);
@@ -145,8 +145,8 @@ int main(int argc, char **argv)
qsort(list2, count, sizeof(list[0]), compare_num);

for (i = 0; i < count; i++)
- fprintf(fout, "%d times:\n%s\n", list2[i].num, list2[i].txt);
-
+ fprintf(fout, "%d times:\n%s\n", list2[i].num, list2[i].txt);
+
fclose(fout);
return 0;
}
--
2.9.3