From 4b7c791e15e921ce5427c531f740df7474c6a1cb Mon Sep 17 00:00:00 2001 From: Neels Hofmeyr Date: Sat, 7 Oct 2017 04:45:01 +0200 Subject: gsm: make osmo_imsi_str_valid() NULL-safe No callers that would pass NULL exist, but let's check against NULL from the start. Fixup for recent change I1e94f5b0717b947d2a7a7d36bacdf04a75cb3522. Change-Id: I111fbf29228929f2cd6ffa06bcb1f69da223224e --- src/gsm/gsm23003.c | 2 ++ 1 file changed, 2 insertions(+) (limited to 'src/gsm') diff --git a/src/gsm/gsm23003.c b/src/gsm/gsm23003.c index 004e20ff..3c09acac 100644 --- a/src/gsm/gsm23003.c +++ b/src/gsm/gsm23003.c @@ -33,6 +33,8 @@ static bool is_n_digits(const char *str, int min_digits, int max_digits) /* Use unsigned char * to avoid a compiler warning of * "error: array subscript has type 'char' [-Werror=char-subscripts]" */ const unsigned char *pos = (const unsigned char *)str; + if (!pos) + return min_digits < 1; for (len = 0; *pos && len < max_digits; len++, pos++) if (!isdigit(*pos)) return false; -- cgit v1.2.3