summaryrefslogtreecommitdiffstats
path: root/lib/libssl/src/apps/rsautl.c
diff options
context:
space:
mode:
authorjsing <jsing@openbsd.org>2014-05-19 12:13:30 +0000
committerjsing <jsing@openbsd.org>2014-05-19 12:13:30 +0000
commit7dc304aadc7dce4fbb9a8f3931d77cdca7c65042 (patch)
treece3a0778eb14bd880d642ec9fc70ef52c8126c0f /lib/libssl/src/apps/rsautl.c
parentNuke some trailing whitespace that snuck in. (diff)
downloadwireguard-openbsd-7dc304aadc7dce4fbb9a8f3931d77cdca7c65042.tar.xz
wireguard-openbsd-7dc304aadc7dce4fbb9a8f3931d77cdca7c65042.zip
It is perfectly valid to call free(NULL), so stop checking for non-NULL
before calling free - of some course parts of the code already did this.
Diffstat (limited to 'lib/libssl/src/apps/rsautl.c')
-rw-r--r--lib/libssl/src/apps/rsautl.c11
1 files changed, 5 insertions, 6 deletions
diff --git a/lib/libssl/src/apps/rsautl.c b/lib/libssl/src/apps/rsautl.c
index 55c89b271bb..08d28a496bd 100644
--- a/lib/libssl/src/apps/rsautl.c
+++ b/lib/libssl/src/apps/rsautl.c
@@ -305,16 +305,15 @@ rsautl_main(int argc, char **argv)
BIO_dump(out, (char *) rsa_out, rsa_outlen);
else
BIO_write(out, rsa_out, rsa_outlen);
+
end:
RSA_free(rsa);
BIO_free(in);
BIO_free_all(out);
- if (rsa_in)
- free(rsa_in);
- if (rsa_out)
- free(rsa_out);
- if (passin)
- free(passin);
+ free(rsa_in);
+ free(rsa_out);
+ free(passin);
+
return ret;
}