diff options
| author |   deraadt <deraadt@openbsd.org> | 2014-05-30 06:22:57 +0000 |
|---|---|---|
| committer | deraadt <deraadt@openbsd.org> | 2014-05-30 06:22:57 +0000 |
| commit | 0b1e30339eeb8f43097c86a51e3f4e526dce5d73 (patch) | |
| tree | 4adc99aff640e367a82c027f2de70634404f66a1 /lib/libcrypto/objects/obj_lib.c | |
| parent | While working on another diff I ended up looking to see why on earth the (diff) | |
| download | wireguard-openbsd-0b1e30339eeb8f43097c86a51e3f4e526dce5d73.tar.xz wireguard-openbsd-0b1e30339eeb8f43097c86a51e3f4e526dce5d73.zip | |
more: no need for null check before free
ok tedu guenther
Diffstat (limited to 'lib/libcrypto/objects/obj_lib.c')
| -rw-r--r-- | lib/libcrypto/objects/obj_lib.c | 12 |
1 files changed, 4 insertions, 8 deletions
diff --git a/lib/libcrypto/objects/obj_lib.c b/lib/libcrypto/objects/obj_lib.c index 57f0b28cc04..25bc8fb5eea 100644 --- a/lib/libcrypto/objects/obj_lib.c +++ b/lib/libcrypto/objects/obj_lib.c @@ -114,14 +114,10 @@ OBJ_dup(const ASN1_OBJECT *o) err: OBJerr(OBJ_F_OBJ_DUP, ERR_R_MALLOC_FAILURE); - if (ln != NULL) - free(ln); - if (sn != NULL) - free(sn); - if (data != NULL) - free(data); - if (r != NULL) - free(r); + free(ln); + free(sn); + free(data); + free(r); return (NULL); } |