On Tue, Apr 22, 2014 at 10:33:55PM -0600, Bob Beck wrote: > Yes, ok > > committed > > > On Wed, Apr 23, 2014 at 03:55:19AM +0200, Dirk Engling wrote: > > Index: x_x509.c > > =================================================================== > > RCS file: /cvs/src/lib/libssl/src/crypto/asn1/x_x509.c,v > > retrieving revision 1.12 > > diff -u -r1.12 x_x509.c > > --- x_x509.c 18 Apr 2014 11:20:32 -0000 1.12 > > +++ x_x509.c 23 Apr 2014 01:54:03 -0000 > > @@ -125,6 +125,7 @@ > > #endif > > if (ret->name != NULL) > > free(ret->name); > > + ret->name = NULL; > > break; > > }
Why not kill the 'if (ret->name != NULL)' check while at it?