From f2beee209bc9a65ff4a0eb85feabe60311741899 Mon Sep 17 00:00:00 2001 From: Daniel Stenberg Date: Sat, 31 Mar 2007 21:35:56 +0000 Subject: [PATCH] Since the str2num() function gets called with the 'nextarg' pointer from within the getparameter a lot, we must check it for NULL before accessing the str data. CID 14 of the coverity.com scan --- src/main.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/main.c b/src/main.c index 72f28e3c9..391288708 100644 --- a/src/main.c +++ b/src/main.c @@ -1273,12 +1273,16 @@ static void cleanarg(char *str) * non-zero on failure, zero on success. * * The string must start with a digit to be valid. + * + * Since this function gets called with the 'nextarg' pointer from within the + * getparameter a lot, we must check it for NULL before accessing the str + * data. */ static int str2num(long *val, char *str) { int retcode = 0; - if(ISDIGIT(*str)) + if(str && ISDIGIT(*str)) *val = atoi(str); else retcode = 1; /* badness */