diff options
author | deraadt <> | 2014-05-30 06:22:57 +0000 |
---|---|---|
committer | deraadt <> | 2014-05-30 06:22:57 +0000 |
commit | abd9b10898e82011a65d8e8fca56bf56532edd69 (patch) | |
tree | 1ff700d79cee9e764329e5c50d7eb8d5109789cc /src/lib/libcrypto/pem | |
parent | 2ff11427d41a52cb126e9892413db066f8ae166a (diff) | |
download | openbsd-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.c | 18 | ||||
-rw-r--r-- | src/lib/libcrypto/pem/pem_seal.c | 6 | ||||
-rw-r--r-- | src/lib/libcrypto/pem/pem_sign.c | 3 |
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 | ||
115 | err: | 115 | err: |
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, | |||
182 | err: | 181 | err: |
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 | ||
101 | err: | 101 | err: |
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 | } |