]> granicus.if.org Git - curl/commitdiff
tool_parsecfg: Fix control flow issue (DEADCODE)
authorSteve Holme <steve_holme@hotmail.com>
Sun, 2 Jun 2019 10:57:19 +0000 (11:57 +0100)
committerSteve Holme <steve_holme@hotmail.com>
Sun, 2 Jun 2019 21:37:05 +0000 (22:37 +0100)
Follow-up to 8144ba38.

Detected by Coverity CID 1445663
Closes #3976

src/tool_parsecfg.c

index 683e25ac18bff43a3cfedf7cf331d2a2e08038a6..621369d458edfb13fe83f9e07490a3db9e8a81ab 100644 (file)
@@ -5,7 +5,7 @@
  *                            | (__| |_| |  _ <| |___
  *                             \___|\___/|_| \_\_____|
  *
- * Copyright (C) 1998 - 2018, Daniel Stenberg, <daniel@haxx.se>, et al.
+ * Copyright (C) 1998 - 2019, Daniel Stenberg, <daniel@haxx.se>, et al.
  *
  * This software is licensed as described in the file COPYING, which
  * you should have received as part of this distribution. The terms
@@ -109,12 +109,12 @@ int parseconfig(const char *filename, struct GlobalConfig *global)
 #endif
   }
 
-  if(file != NULL)  /* WIN32: no need to fopen() again */
-    ;
-  else if(strcmp(filename, "-"))
-    file = fopen(filename, FOPEN_READTEXT);
-  else
-    file = stdin;
+  if(!file) { /* WIN32: no need to fopen() again */
+    if(strcmp(filename, "-"))
+      file = fopen(filename, FOPEN_READTEXT);
+    else
+      file = stdin;
+  }
 
   if(file) {
     char *line;