summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorNeels Hofmeyr <neels@hofmeyr.de>2019-02-11 21:24:40 +0100
committerNeels Hofmeyr <nhofmeyr@sysmocom.de>2019-02-15 03:01:54 +0000
commita0331edcefb46a2f6451d8000b48f17ea7342a53 (patch)
tree8678f4e1ea38adecb25e326a9145a93d5115217a
parent450f5ac9105ffeccbebc52d9d61bbb1b0520cfad (diff)
get_value_string(): guard against NULL
get_value_string() conveniently prints the value number to a static buffer if it is unknown in a value_string array. Do the same if the value_string array pointer itself is NULL. If a value string array is user supplied and might be NULL, one could add a separate NULL check around it; but by making get_value_string() itself guard against NULL, another static char buffer to print the value number is avoided. Change-Id: Ie640e9258a959da8f4f9089478de993509853997
-rw-r--r--src/utils.c3
1 files changed, 3 insertions, 0 deletions
diff --git a/src/utils.c b/src/utils.c
index 0b2ed31d..2d5bcb0e 100644
--- a/src/utils.c
+++ b/src/utils.c
@@ -73,6 +73,9 @@ const char *get_value_string_or_null(const struct value_string *vs,
{
int i;
+ if (!vs)
+ return NULL;
+
for (i = 0;; i++) {
if (vs[i].value == 0 && vs[i].str == NULL)
break;