|
 |
e0bf0f |
From d6149aaad2a72a8f000283015f6e381bb2821ee2 Mon Sep 17 00:00:00 2001
|
|
|
e0bf0f |
From: =?UTF-8?q?Petr=20Men=C5=A1=C3=ADk?= <pemensik@redhat.com>
|
|
|
e0bf0f |
Date: Thu, 7 Jan 2021 16:08:40 +0100
|
|
|
e0bf0f |
Subject: [PATCH 1/3] Invalid dereference
|
|
|
e0bf0f |
|
|
|
e0bf0f |
libmy/argv.c:3212: var_deref_model: Passing null pointer "queue_list" to "do_list", which dereferences it
|
|
|
e0bf0f |
libmy/argv.c:3204: var_deref_model: Passing null pointer "queue_list" to "do_list", which dereferences it.
|
|
|
e0bf0f |
|
|
|
e0bf0f |
Workaround to possibility no arguments is received
|
|
|
e0bf0f |
|
|
|
e0bf0f |
Usually at least one arg is always passed in argv - program name. Do not
|
|
|
e0bf0f |
dereference null queue_list in unlikely case no parameter in argv.
|
|
|
e0bf0f |
---
|
|
|
e0bf0f |
libmy/argv.c | 45 +++++++++++++++++++++++----------------------
|
|
|
e0bf0f |
1 file changed, 23 insertions(+), 22 deletions(-)
|
|
|
e0bf0f |
|
|
|
e0bf0f |
diff --git a/libmy/argv.c b/libmy/argv.c
|
|
|
e0bf0f |
index 6c64906..c3aadfe 100644
|
|
|
e0bf0f |
--- a/libmy/argv.c
|
|
|
e0bf0f |
+++ b/libmy/argv.c
|
|
|
e0bf0f |
@@ -3197,28 +3197,29 @@ int argv_process_no_env(argv_t *args, const int arg_n, char **argv)
|
|
|
e0bf0f |
}
|
|
|
e0bf0f |
queue_head = 0;
|
|
|
e0bf0f |
queue_tail = 0;
|
|
|
e0bf0f |
- }
|
|
|
e0bf0f |
-
|
|
|
e0bf0f |
- /* do the env args before? */
|
|
|
e0bf0f |
- if (argv_process_env_b && (! argv_env_after_b) && env_vect_p != NULL) {
|
|
|
e0bf0f |
- do_list(args, env_n, env_vect_p, queue_list, &queue_head, &queue_tail,
|
|
|
e0bf0f |
- &okay_b);
|
|
|
e0bf0f |
- free(env_vect_p);
|
|
|
e0bf0f |
- free(environ_p);
|
|
|
e0bf0f |
- env_vect_p = NULL;
|
|
|
e0bf0f |
- }
|
|
|
e0bf0f |
-
|
|
|
e0bf0f |
- /* do the external args */
|
|
|
e0bf0f |
- do_list(args, arg_n - 1, argv + 1, queue_list, &queue_head, &queue_tail,
|
|
|
e0bf0f |
- &okay_b);
|
|
|
e0bf0f |
+
|
|
|
e0bf0f |
+ /* do the env args before? */
|
|
|
e0bf0f |
+ if (argv_process_env_b && (! argv_env_after_b) && env_vect_p != NULL) {
|
|
|
e0bf0f |
+ do_list(args, env_n, env_vect_p, queue_list, &queue_head, &queue_tail,
|
|
|
e0bf0f |
+ &okay_b);
|
|
|
e0bf0f |
+ free(env_vect_p);
|
|
|
e0bf0f |
+ free(environ_p);
|
|
|
e0bf0f |
+ env_vect_p = NULL;
|
|
|
e0bf0f |
+ }
|
|
|
e0bf0f |
+
|
|
|
e0bf0f |
+ /* do the external args */
|
|
|
e0bf0f |
+ if (arg_n > 0)
|
|
|
e0bf0f |
+ do_list(args, arg_n - 1, argv + 1, queue_list, &queue_head, &queue_tail,
|
|
|
e0bf0f |
+ &okay_b);
|
|
|
e0bf0f |
|
|
|
e0bf0f |
- /* DO the env args after? */
|
|
|
e0bf0f |
- if (argv_process_env_b && argv_env_after_b && env_vect_p != NULL) {
|
|
|
e0bf0f |
- do_list(args, env_n, env_vect_p, queue_list, &queue_head, &queue_tail,
|
|
|
e0bf0f |
- &okay_b);
|
|
|
e0bf0f |
- free(env_vect_p);
|
|
|
e0bf0f |
- free(environ_p);
|
|
|
e0bf0f |
- env_vect_p = NULL;
|
|
|
e0bf0f |
+ /* DO the env args after? */
|
|
|
e0bf0f |
+ if (argv_process_env_b && argv_env_after_b && env_vect_p != NULL) {
|
|
|
e0bf0f |
+ do_list(args, env_n, env_vect_p, queue_list, &queue_head, &queue_tail,
|
|
|
e0bf0f |
+ &okay_b);
|
|
|
e0bf0f |
+ free(env_vect_p);
|
|
|
e0bf0f |
+ free(environ_p);
|
|
|
e0bf0f |
+ env_vect_p = NULL;
|
|
|
e0bf0f |
+ }
|
|
|
e0bf0f |
}
|
|
|
e0bf0f |
|
|
|
e0bf0f |
/* make sure the XOR and MAND args and argument-options are okay */
|
|
|
e0bf0f |
@@ -3233,7 +3234,7 @@ int argv_process_no_env(argv_t *args, const int arg_n, char **argv)
|
|
|
e0bf0f |
}
|
|
|
e0bf0f |
|
|
|
e0bf0f |
/* if we allocated the space then free it */
|
|
|
e0bf0f |
- if (arg_n > 0) {
|
|
|
e0bf0f |
+ if (queue_list) {
|
|
|
e0bf0f |
free(queue_list);
|
|
|
e0bf0f |
}
|
|
|
e0bf0f |
|
|
|
e0bf0f |
--
|
|
|
e0bf0f |
2.26.3
|
|
|
e0bf0f |
|