src/rrd_graph_helper.c: fix Coverity CID#13675 Dereference before null check

        CWE-476 / https://cwe.mitre.org/data/definitions/476.html

diff --git a/src/rrd_graph_helper.c b/src/rrd_graph_helper.c
index 0f518eece1797ba23f0c52d87f1f1aff73f17dcb..3a6124824949835619043c0d0e330fee2fadc465 100644 (file)
--- a/src/rrd_graph_helper.c
+++ b/src/rrd_graph_helper.c
@@ -140,20 +140,19 @@ int getMappedKeyValueArgument(const char* key,int flag, parsedargs_t* pa,
 }
 
 int getLong(const char* v,long *val,char**extra,int base) {
+  if (extra == NULL)
+    return 0;
+
   /* try to execute the parser */
   /* NOTE that this may be a bit different from the original parser */
   *extra=NULL;
   *val=strtol(v,extra,base);
   /* and error handling */
-  if (extra==NULL) {
-    return 0;
-  } else {
-    if (*extra==v) {
-      return -1; /* failed miserably */
-    }  else {
-      if ((*extra)[0]==0) { return 0; }
-      return 1; /* got extra bytes */
-    }
+  if (*extra==v) {
+    return -1; /* failed miserably */
+  }  else {
+    if ((*extra)[0]==0) { return 0; }
+    return 1; /* got extra bytes */
   }
   /* not found, so return error */
   return -2;