Two minor code cleanups and one NULL deref on error fix.
authorWerner Koch <wk@gnupg.org>
Tue, 15 Apr 2014 14:40:48 +0000 (16:40 +0200)
committerWerner Koch <wk@gnupg.org>
Wed, 16 Apr 2014 08:18:17 +0000 (10:18 +0200)
* common/estream.c (es_freopen): Remove useless check for STREAM.
* kbx/keybox-blob.c (_keybox_create_x509_blob): Remove useless check
for BLOB.
* tools/sockprox.c (run_proxy): Do not fclose(NULL).
--

Found by Hans-Christoph Steiner with cppcheck.

common/estream.c
kbx/keybox-blob.c
tools/sockprox.c

index be791a8..6272c81 100644 (file)
@@ -3101,7 +3101,7 @@ es_freopen (const char *ES__RESTRICT path, const char *ES__RESTRICT mode,
        }
       else
         {
-          if (stream && path)
+          if (path)
             fname_set_internal (stream, path, 1);
           ESTREAM_UNLOCK (stream);
         }
index a38f991..ebbcdf8 100644 (file)
@@ -948,12 +948,12 @@ _keybox_create_x509_blob (KEYBOXBLOB *r_blob, ksba_cert_t cert,
  leave:
   release_kid_list (blob->temp_kids);
   blob->temp_kids = NULL;
-  if (blob && names)
+  if (names)
     {
       for (i=0; i < blob->nuids; i++)
         xfree (names[i]);
+      xfree (names);
     }
-  xfree (names);
   if (rc)
     {
       _keybox_release_blob (blob);
index 9ec89ce..3593598 100644 (file)
@@ -443,7 +443,8 @@ run_proxy (void)
  out:
 
   pthread_attr_destroy (&thread_attr);
-  fclose (protocol_file);      /* FIXME, err checking.  */
+  if (protocol_file)
+    fclose (protocol_file);    /* FIXME, err checking.  */
 
   return err;
 }