|
 |
b81b3d |
From 1499d3e2715bad67588b5c0b6c02865eeb65aa16 Mon Sep 17 00:00:00 2001
|
|
|
b81b3d |
From: =?UTF-8?q?Petr=20Men=C5=A1=C3=ADk?= <pemensik@redhat.com>
|
|
|
b81b3d |
Date: Fri, 8 Jan 2021 17:43:03 +0100
|
|
|
b81b3d |
Subject: [PATCH 3/3] Possible resource leak fix
|
|
|
b81b3d |
|
|
|
b81b3d |
34. fstrm-0.6.0/libmy/argv.c:2238: alloc_fn: Storage is returned from allocation function "realloc".
|
|
|
b81b3d |
35. fstrm-0.6.0/libmy/argv.c:2238: var_assign: Assigning: "argv" = storage returned from "realloc(argv, 8UL * max)".
|
|
|
b81b3d |
37. fstrm-0.6.0/libmy/argv.c:2254: var_assign: Assigning: "argv_p" = "argv".
|
|
|
b81b3d |
47. fstrm-0.6.0/libmy/argv.c:2229: leaked_storage: Variable "argv_p" going out of scope leaks the storage it points to.
|
|
|
b81b3d |
48. fstrm-0.6.0/libmy/argv.c:2229: leaked_storage: Variable "argv" going out of scope leaks the storage it points to.
|
|
|
b81b3d |
---
|
|
|
b81b3d |
libmy/argv.c | 5 +++--
|
|
|
b81b3d |
1 file changed, 3 insertions(+), 2 deletions(-)
|
|
|
b81b3d |
|
|
|
b81b3d |
diff --git a/libmy/argv.c b/libmy/argv.c
|
|
|
b81b3d |
index 16dca73..0b28026 100644
|
|
|
b81b3d |
--- a/libmy/argv.c
|
|
|
b81b3d |
+++ b/libmy/argv.c
|
|
|
b81b3d |
@@ -2226,7 +2226,7 @@ static void file_args(const char *path, argv_t *grid,
|
|
|
b81b3d |
*argv_p = string_copy(line);
|
|
|
b81b3d |
if (*argv_p == NULL) {
|
|
|
b81b3d |
*okay_bp = ARGV_FALSE;
|
|
|
b81b3d |
- return;
|
|
|
b81b3d |
+ goto cleanup;
|
|
|
b81b3d |
}
|
|
|
b81b3d |
|
|
|
b81b3d |
argv_p++;
|
|
|
b81b3d |
@@ -2257,7 +2257,8 @@ static void file_args(const char *path, argv_t *grid,
|
|
|
b81b3d |
|
|
|
b81b3d |
/* now do the list */
|
|
|
b81b3d |
do_list(grid, arg_c, argv, queue_list, queue_head_p, queue_tail_p, okay_bp);
|
|
|
b81b3d |
-
|
|
|
b81b3d |
+
|
|
|
b81b3d |
+cleanup:
|
|
|
b81b3d |
/* now free up the list */
|
|
|
b81b3d |
for (argv_p = argv; argv_p < argv + arg_c; argv_p++) {
|
|
|
b81b3d |
free(*argv_p);
|
|
|
b81b3d |
--
|
|
|
b81b3d |
2.26.3
|
|
|
b81b3d |
|