summaryrefslogtreecommitdiff
path: root/src/lib/libcrypto/pem
diff options
context:
space:
mode:
authorderaadt <>2014-05-30 06:22:57 +0000
committerderaadt <>2014-05-30 06:22:57 +0000
commitabd9b10898e82011a65d8e8fca56bf56532edd69 (patch)
tree1ff700d79cee9e764329e5c50d7eb8d5109789cc /src/lib/libcrypto/pem
parent2ff11427d41a52cb126e9892413db066f8ae166a (diff)
downloadopenbsd-abd9b10898e82011a65d8e8fca56bf56532edd69.tar.gz
openbsd-abd9b10898e82011a65d8e8fca56bf56532edd69.tar.bz2
openbsd-abd9b10898e82011a65d8e8fca56bf56532edd69.zip
more: no need for null check before free
ok tedu guenther
Diffstat (limited to 'src/lib/libcrypto/pem')
-rw-r--r--src/lib/libcrypto/pem/pem_info.c18
-rw-r--r--src/lib/libcrypto/pem/pem_seal.c6
-rw-r--r--src/lib/libcrypto/pem/pem_sign.c3
3 files changed, 9 insertions, 18 deletions
diff --git a/src/lib/libcrypto/pem/pem_info.c b/src/lib/libcrypto/pem/pem_info.c
index 40600e30f3..7605c7653c 100644
--- a/src/lib/libcrypto/pem/pem_info.c
+++ b/src/lib/libcrypto/pem/pem_info.c
@@ -256,12 +256,9 @@ start:
256 } else { 256 } else {
257 /* unknown */ 257 /* unknown */
258 } 258 }
259 if (name != NULL) 259 free(name);
260 free(name); 260 free(header);
261 if (header != NULL) 261 free(data);
262 free(header);
263 if (data != NULL)
264 free(data);
265 name = NULL; 262 name = NULL;
266 header = NULL; 263 header = NULL;
267 data = NULL; 264 data = NULL;
@@ -291,12 +288,9 @@ err:
291 ret = NULL; 288 ret = NULL;
292 } 289 }
293 290
294 if (name != NULL) 291 free(name);
295 free(name); 292 free(header);
296 if (header != NULL) 293 free(data);
297 free(header);
298 if (data != NULL)
299 free(data);
300 return (ret); 294 return (ret);
301} 295}
302 296
diff --git a/src/lib/libcrypto/pem/pem_seal.c b/src/lib/libcrypto/pem/pem_seal.c
index a7b9379223..e6e73dd23d 100644
--- a/src/lib/libcrypto/pem/pem_seal.c
+++ b/src/lib/libcrypto/pem/pem_seal.c
@@ -113,8 +113,7 @@ PEM_SealInit(PEM_ENCODE_SEAL_CTX *ctx, EVP_CIPHER *type, EVP_MD *md_type,
113 ret = npubk; 113 ret = npubk;
114 114
115err: 115err:
116 if (s != NULL) 116 free(s);
117 free(s);
118 OPENSSL_cleanse(key, EVP_MAX_KEY_LENGTH); 117 OPENSSL_cleanse(key, EVP_MAX_KEY_LENGTH);
119 return (ret); 118 return (ret);
120} 119}
@@ -182,8 +181,7 @@ PEM_SealFinal(PEM_ENCODE_SEAL_CTX *ctx, unsigned char *sig, int *sigl,
182err: 181err:
183 EVP_MD_CTX_cleanup(&ctx->md); 182 EVP_MD_CTX_cleanup(&ctx->md);
184 EVP_CIPHER_CTX_cleanup(&ctx->cipher); 183 EVP_CIPHER_CTX_cleanup(&ctx->cipher);
185 if (s != NULL) 184 free(s);
186 free(s);
187 return (ret); 185 return (ret);
188} 186}
189#endif 187#endif
diff --git a/src/lib/libcrypto/pem/pem_sign.c b/src/lib/libcrypto/pem/pem_sign.c
index 8a6bcc8d8d..bfe7823e3d 100644
--- a/src/lib/libcrypto/pem/pem_sign.c
+++ b/src/lib/libcrypto/pem/pem_sign.c
@@ -100,7 +100,6 @@ PEM_SignFinal(EVP_MD_CTX *ctx, unsigned char *sigret, unsigned int *siglen,
100 100
101err: 101err:
102 /* ctx has been zeroed by EVP_SignFinal() */ 102 /* ctx has been zeroed by EVP_SignFinal() */
103 if (m != NULL) 103 free(m);
104 free(m);
105 return (ret); 104 return (ret);
106} 105}