]> granicus.if.org Git - neomutt/commitdiff
remove unnecessary alloc checks
authorRichard Russon <rich@flatcap.org>
Tue, 21 Nov 2017 16:59:48 +0000 (16:59 +0000)
committerRichard Russon <rich@flatcap.org>
Sun, 26 Nov 2017 22:43:34 +0000 (22:43 +0000)
conn/ssl_gnutls.c
imap/util.c

index b64f9e0878b72260c31bd064002bb2c0ab54783a..4e3e8811e1cc0f8ae894fab79bf74197f85c8dee 100644 (file)
@@ -992,12 +992,8 @@ static void tls_get_client_cert(struct Connection *conn)
   /* get length of DN */
   dnlen = 0;
   gnutls_x509_crt_get_dn(clientcrt, NULL, &dnlen);
-  dn = calloc(1, dnlen);
-  if (!dn)
-  {
-    mutt_debug(1, "could not allocate DN\n");
-    goto err_crt;
-  }
+  dn = mutt_mem_calloc(1, dnlen);
+
   gnutls_x509_crt_get_dn(clientcrt, dn, &dnlen);
   mutt_debug(2, "client certificate DN: %s\n", dn);
 
index 1e93176dca86abef2c2bbc8581a46827c364f080..db3da95d871ec9e6a236d473f5ed3c63d1d68db2 100644 (file)
@@ -622,11 +622,6 @@ struct ImapData *imap_new_idata(void)
 
   idata->cmdslots = ImapPipelineDepth + 2;
   idata->cmds = mutt_mem_calloc(idata->cmdslots, sizeof(*idata->cmds));
-  if (!idata->cmds)
-  {
-    mutt_buffer_free(&idata->cmdbuf);
-    FREE(&idata);
-  }
 
   STAILQ_INIT(&idata->flags);
   STAILQ_INIT(&idata->mboxcache);