-coverity, cleanup
authorMartin Schanzenbach <mschanzenbach@posteo.de>
Mon, 11 Jun 2012 12:35:36 +0000 (12:35 +0000)
committerMartin Schanzenbach <mschanzenbach@posteo.de>
Mon, 11 Jun 2012 12:35:36 +0000 (12:35 +0000)
src/gns/gnunet-gns-proxy.c
src/gns/nss/nss_gns_query.c

index 9365295e4eadf7a6f79c363b59c497f5bd4939df..fcfe2c76285b5a9806e23f9033a432c881b242c7 100644 (file)
@@ -1040,10 +1040,12 @@ process_get_authority (void *cls,
                 "Get authority failed!\n");
     strcpy (ctask->authority, "");
   }
-
-  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-              "Get authority yielded %s\n", auth_name);
-  strcpy (ctask->authority, auth_name);
+  else
+  {
+    GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+                "Get authority yielded %s\n", auth_name);
+    strcpy (ctask->authority, auth_name);
+  }
 
   GNUNET_GNS_lookup_zone (gns_handle,
                           ctask->host,
@@ -1615,11 +1617,9 @@ generate_gns_certificate (const char *name)
 
   int ret;
   unsigned int serial;
-  unsigned int bits;
   size_t key_buf_size;
   size_t cert_buf_size;
   gnutls_x509_crt_t request;
-  gnutls_x509_privkey_t rsa;
   time_t etime;
   struct tm *tm_data;
 
index ab88d22cb2634b4962f0621b2a360a290def1956..1eadff0ce4712fdbde559a4766ce46705df105ef 100644 (file)
@@ -46,19 +46,31 @@ int gns_resolve_name(int af, const char *name, struct userdata *u)
         line[strlen(line)-1] = '\0';
         if (af == AF_INET)
         {
-          inet_pton(af, line, &(u->data.ipv4[u->count++]));
-          u->data_len += sizeof(ipv4_address_t);
+          if (inet_pton(af, line, &(u->data.ipv4[u->count++])))
+            u->data_len += sizeof(ipv4_address_t);
+          else
+          {
+            fclose (p);
+            free (cmd);
+            return -1;
+          }
         }
         else if ((af == AF_INET6))
         {
-          inet_pton(af, line, &(u->data.ipv6[u->count++]));
-          u->data_len += sizeof(ipv6_address_t);
+          if (inet_pton(af, line, &(u->data.ipv6[u->count++])))
+            u->data_len += sizeof(ipv6_address_t);
+          else
+          {
+            fclose (p);
+            free (cmd);
+            return -1;
+          }
         }
       }
     }
+    fclose (p);
   }
-  fclose(p);
-  free(cmd);
+  free (cmd);
 
   return 0;