src/pgp_sequoia.c
branchENGINE-524
changeset 3841 f9d1e79ae49b
parent 3840 bfef390b48e2
parent 3801 8854f7fd79a8
     1.1 --- a/src/pgp_sequoia.c	Sat Jun 01 13:18:11 2019 +0200
     1.2 +++ b/src/pgp_sequoia.c	Thu Jun 13 17:25:55 2019 +0200
     1.3 @@ -844,15 +844,12 @@
     1.4  
     1.5      free(email);
     1.6      free(name);
     1.7 -    if (user_id_iter)
     1.8 -        pgp_user_id_binding_iter_free(user_id_iter);
     1.9 -    if (key_iter)
    1.10 -        pgp_tpk_key_iter_free(key_iter);
    1.11 +    pgp_user_id_binding_iter_free(user_id_iter);
    1.12 +    pgp_tpk_key_iter_free(key_iter);
    1.13      if (stmt)
    1.14        sqlite3_reset(stmt);
    1.15      free(tsk_buffer);
    1.16 -    if (tpk)
    1.17 -        pgp_tpk_free(tpk);
    1.18 +    pgp_tpk_free(tpk);
    1.19      free(fpr);
    1.20      pgp_fingerprint_free(pgp_fpr);
    1.21  
    1.22 @@ -1004,13 +1001,10 @@
    1.23          cookie->decrypted = 1;
    1.24  
    1.25      eol:
    1.26 -        if (sk)
    1.27 -            pgp_session_key_free (sk);
    1.28 +        pgp_session_key_free (sk);
    1.29          free(keyid_str);
    1.30 -        if (key_iter)
    1.31 -            pgp_tpk_key_iter_free(key_iter);
    1.32 -        if (tpk)
    1.33 -            pgp_tpk_free(tpk);
    1.34 +        pgp_tpk_key_iter_free(key_iter);
    1.35 +        pgp_tpk_free(tpk);
    1.36      }
    1.37  
    1.38      // Consider wildcard recipients.
    1.39 @@ -1083,11 +1077,9 @@
    1.40              key_iter = NULL;
    1.41          }
    1.42      eol2:
    1.43 -        if (sk)
    1.44 -            pgp_session_key_free (sk);
    1.45 +        pgp_session_key_free (sk);
    1.46          free(keyid_str);
    1.47 -        if (key_iter)
    1.48 -            pgp_tpk_key_iter_free(key_iter);
    1.49 +        pgp_tpk_key_iter_free(key_iter);
    1.50      }
    1.51  
    1.52      if (tsks) {
    1.53 @@ -1421,12 +1413,9 @@
    1.54          free(*ptext);
    1.55      }
    1.56  
    1.57 -    if (reader)
    1.58 -        pgp_reader_free(reader);
    1.59 -    if (decryptor)
    1.60 -        pgp_reader_free(decryptor);
    1.61 -    if (writer)
    1.62 -        pgp_writer_free(writer);
    1.63 +    pgp_reader_free(reader);
    1.64 +    pgp_reader_free(decryptor);
    1.65 +    pgp_writer_free(writer);
    1.66  
    1.67      T("-> %s", pEp_status_to_string(status));
    1.68      return status;
    1.69 @@ -1520,12 +1509,9 @@
    1.70          free_stringlist(cookie.signer_keylist);
    1.71      }
    1.72  
    1.73 -    if (verifier)
    1.74 -        pgp_reader_free(verifier);
    1.75 -    if (reader)
    1.76 -        pgp_reader_free(reader);
    1.77 -    if (dsig_reader)
    1.78 -        pgp_reader_free(dsig_reader);
    1.79 +    pgp_reader_free(verifier);
    1.80 +    pgp_reader_free(reader);
    1.81 +    pgp_reader_free(dsig_reader);
    1.82  
    1.83      T("-> %s", pEp_status_to_string(status));
    1.84      return status;
    1.85 @@ -1607,14 +1593,10 @@
    1.86      (*stext)[*ssize] = 0;
    1.87  
    1.88   out:
    1.89 -    if (signer)
    1.90 -        pgp_signer_free (signer);
    1.91 -    if (signing_keypair)
    1.92 -        pgp_key_pair_free (signing_keypair);
    1.93 -    if (iter)
    1.94 -        pgp_tpk_key_iter_free (iter);
    1.95 -    if (signer_tpk)
    1.96 -        pgp_tpk_free(signer_tpk);
    1.97 +    pgp_signer_free (signer);
    1.98 +    pgp_key_pair_free (signing_keypair);
    1.99 +    pgp_tpk_key_iter_free (iter);
   1.100 +    pgp_tpk_free(signer_tpk);
   1.101  
   1.102      T("(%s)-> %s", fpr, pEp_status_to_string(status));
   1.103      return status;
   1.104 @@ -1728,14 +1710,10 @@
   1.105      (*ctext)[*csize] = 0;
   1.106  
   1.107   out:
   1.108 -    if (signer)
   1.109 -        pgp_signer_free (signer);
   1.110 -    if (signing_keypair)
   1.111 -        pgp_key_pair_free (signing_keypair);
   1.112 -    if (iter)
   1.113 -        pgp_tpk_key_iter_free (iter);
   1.114 -    if (signer_tpk)
   1.115 -        pgp_tpk_free(signer_tpk);
   1.116 +    pgp_signer_free (signer);
   1.117 +    pgp_key_pair_free (signing_keypair);
   1.118 +    pgp_tpk_key_iter_free (iter);
   1.119 +    pgp_tpk_free(signer_tpk);
   1.120  
   1.121      for (int i = 0; i < keys_count; i ++)
   1.122          pgp_tpk_free(keys[i]);
   1.123 @@ -1787,7 +1765,7 @@
   1.124      size_t userid_len = 0;
   1.125      const uint8_t *raw = pgp_user_id_value(userid_packet, &userid_len);
   1.126  
   1.127 -    // Null terminate it.
   1.128 +    // NUL terminate it.
   1.129      userid = malloc(userid_len + 1);
   1.130      if (!userid)
   1.131          ERROR_OUT(NULL, PEP_OUT_OF_MEMORY, "out of memory");
   1.132 @@ -1821,14 +1799,11 @@
   1.133      fpr = NULL;
   1.134  
   1.135   out:
   1.136 -    if (pgp_fpr)
   1.137 -        pgp_fingerprint_free(pgp_fpr);
   1.138 +    pgp_fingerprint_free(pgp_fpr);
   1.139      free(fpr);
   1.140 -    if (tpk)
   1.141 -        pgp_tpk_free(tpk);
   1.142 +    pgp_tpk_free(tpk);
   1.143      free(userid);
   1.144 -    if (userid_packet)
   1.145 -        pgp_packet_free(userid_packet);
   1.146 +    pgp_packet_free(userid_packet);
   1.147  
   1.148      T("-> %s", pEp_status_to_string(status));
   1.149      return status;
   1.150 @@ -1982,8 +1957,7 @@
   1.151      }
   1.152  
   1.153   out:
   1.154 -    if (parser)
   1.155 -        pgp_tpk_parser_free(parser);
   1.156 +    pgp_tpk_parser_free(parser);
   1.157  
   1.158      T("-> %s", pEp_status_to_string(status));
   1.159      return status;
   1.160 @@ -2055,12 +2029,12 @@
   1.161      return status;
   1.162  }
   1.163  
   1.164 -char* _undot_address(const char* address) {
   1.165 +static char *_undot_address(const char* address) {
   1.166      if (!address)
   1.167          return NULL;
   1.168  
   1.169      int addr_len = strlen(address);
   1.170 -    const char* at = strstr(address, "@");
   1.171 +    const char* at = memchr(address, '@', addr_len);
   1.172  
   1.173      if (!at)
   1.174          at = address + addr_len;
   1.175 @@ -2209,10 +2183,8 @@
   1.176      }
   1.177  
   1.178   out:
   1.179 -    if (tpk)
   1.180 -        pgp_tpk_free(tpk);
   1.181 -    if (fpr)
   1.182 -        pgp_fingerprint_free(fpr);
   1.183 +    pgp_tpk_free(tpk);
   1.184 +    pgp_fingerprint_free(fpr);
   1.185  
   1.186      if (status == PEP_KEY_NOT_FOUND)
   1.187          status = PEP_STATUS_OK;
   1.188 @@ -2357,8 +2329,7 @@
   1.189      }
   1.190  
   1.191   out:
   1.192 -    if (tpk)
   1.193 -        pgp_tpk_free(tpk);
   1.194 +    pgp_tpk_free(tpk);
   1.195  
   1.196      T("(%s) -> %s", fpr, pEp_comm_type_to_string(*comm_type));
   1.197      return status;
   1.198 @@ -2418,14 +2389,10 @@
   1.199      ERROR_OUT(NULL, status, "Saving %s", fpr);
   1.200  
   1.201   out:
   1.202 -    if (signer)
   1.203 -        pgp_signer_free (signer);
   1.204 -    if (keypair)
   1.205 -        pgp_key_pair_free (keypair);
   1.206 -    if (iter)
   1.207 -        pgp_tpk_key_iter_free (iter);
   1.208 -    if (tpk)
   1.209 -        pgp_tpk_free(tpk);
   1.210 +    pgp_signer_free (signer);
   1.211 +    pgp_key_pair_free (keypair);
   1.212 +    pgp_tpk_key_iter_free (iter);
   1.213 +    pgp_tpk_free(tpk);
   1.214  
   1.215      T("(%s) -> %s", fpr, pEp_status_to_string(status));
   1.216      return status;
   1.217 @@ -2479,14 +2446,10 @@
   1.218      ERROR_OUT(NULL, status, "Saving %s", fpr);
   1.219  
   1.220   out:
   1.221 -    if (signer)
   1.222 -        pgp_signer_free (signer);
   1.223 -    if (keypair)
   1.224 -        pgp_key_pair_free (keypair);
   1.225 -    if (iter)
   1.226 -        pgp_tpk_key_iter_free (iter);
   1.227 -    if (tpk)
   1.228 -        pgp_tpk_free(tpk);
   1.229 +    pgp_signer_free (signer);
   1.230 +    pgp_key_pair_free (keypair);
   1.231 +    pgp_tpk_key_iter_free (iter);
   1.232 +    pgp_tpk_free(tpk);
   1.233  
   1.234      T("(%s) -> %s", fpr, pEp_status_to_string(status));
   1.235      return status;
   1.236 @@ -2543,8 +2506,7 @@
   1.237      *expired = !(can_encrypt && can_sign && can_certify);
   1.238  
   1.239   out:
   1.240 -    if (tpk)
   1.241 -        pgp_tpk_free(tpk);
   1.242 +    pgp_tpk_free(tpk);
   1.243      T("(%s) -> %s", fpr, pEp_status_to_string(status));
   1.244      return status;
   1.245  }