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 | } |
