Hi tech@, 

Submitting patch to simplify code around free(3) in libssl.
free() already handles the NULL case. 

Does this look ok?

- Michael


Index: t1_enc.c
===================================================================
RCS file: /cvs/src/lib/libssl/src/ssl/t1_enc.c,v
retrieving revision 1.26
diff -u -r1.26 t1_enc.c
--- t1_enc.c    21 Apr 2014 16:34:43 -0000      1.26
+++ t1_enc.c    24 Apr 2014 05:29:52 -0000
@@ -1141,10 +1141,8 @@
        SSLerr(SSL_F_TLS1_EXPORT_KEYING_MATERIAL, ERR_R_MALLOC_FAILURE);
        rv = 0;
 ret:
-       if (buff != NULL)
-               free(buff);
-       if (val != NULL)
-               free(val);
+       free(buff);
+       free(val);
        return (rv);
 }
 

Reply via email to