diff options
author | Rich Salz <rsalz@openssl.org> | 2015-05-01 20:37:16 +0200 |
---|---|---|
committer | Rich Salz <rsalz@openssl.org> | 2015-05-01 20:37:16 +0200 |
commit | 25aaa98aa249d26391c1994d2de449562c8b8b99 (patch) | |
tree | 6f83efd87fa9fd832e8a456e9686143a29f1dab3 /apps/pkcs12.c | |
parent | Remove goto inside an if(0) block (diff) | |
download | openssl-25aaa98aa249d26391c1994d2de449562c8b8b99.tar.xz openssl-25aaa98aa249d26391c1994d2de449562c8b8b99.zip |
free NULL cleanup -- coda
After the finale, the "real" final part. :) Do a recursive grep with
"-B1 -w [a-zA-Z0-9_]*_free" to see if any of the preceeding lines are
an "if NULL" check that can be removed.
Reviewed-by: Tim Hudson <tjh@openssl.org>
Diffstat (limited to 'apps/pkcs12.c')
-rw-r--r-- | apps/pkcs12.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/apps/pkcs12.c b/apps/pkcs12.c index 4c62392e0b..0a4ee3ed10 100644 --- a/apps/pkcs12.c +++ b/apps/pkcs12.c @@ -554,8 +554,7 @@ int pkcs12_main(int argc, char **argv) app_RAND_write_file(NULL); BIO_free(in); BIO_free_all(out); - if (canames) - sk_OPENSSL_STRING_free(canames); + sk_OPENSSL_STRING_free(canames); OPENSSL_free(passin); OPENSSL_free(passout); return (ret); |