Good catch, thanks!  I just applied a change.

In message <[EMAIL PROTECTED]> on Wed, 13 Nov 2002 20:04:19 
+0100, Nils Larsch <[EMAIL PROTECTED]> said:

nlarsch> [EMAIL PROTECTED] wrote:
nlarsch> ...
nlarsch> >   Index: t_x509.c
nlarsch> >   ===================================================================
nlarsch> >   RCS file: /e/openssl/cvs/openssl/crypto/asn1/t_x509.c,v
nlarsch> >   retrieving revision 1.31
nlarsch> >   retrieving revision 1.32
nlarsch> >   diff -u -r1.31 -r1.32
nlarsch> >   --- t_x509.c       2002/08/07 10:49:22     1.31
nlarsch> >   +++ t_x509.c       2002/11/13 15:42:14     1.32
nlarsch> >   @@ -444,15 +444,17 @@
nlarsch> >
nlarsch> >    int X509_NAME_print(BIO *bp, X509_NAME *name, int obase)
nlarsch> >      {
nlarsch> >   -  char *s,*c;
nlarsch> >   +  char *s,*c,*b;
nlarsch> >      int ret=0,l,ll,i,first=1;
nlarsch> >   -  char buf[256];
nlarsch> >
nlarsch> >      ll=80-2-obase;
nlarsch> >
nlarsch> >   -  s=X509_NAME_oneline(name,buf,256);
nlarsch> >   +  b=s=X509_NAME_oneline(name,NULL,0);
nlarsch> >      if (!*s)
nlarsch> >   +          {
nlarsch> >   +          free(b);
nlarsch>                 ^^^^^^
nlarsch> Shouldn't it be OPENSSL_free() ?
nlarsch> 
nlarsch> >              return 1;
nlarsch> >   +          }
nlarsch> >      s++; /* skip the first slash */
nlarsch> >
nlarsch> >      l=ll;
nlarsch> >   @@ -508,6 +510,7 @@
nlarsch> >    err:
nlarsch> >              X509err(X509_F_X509_NAME_PRINT,ERR_R_BUF_LIB);
nlarsch> >              }
nlarsch> >   +  free(b);
nlarsch>         ^^^^^^
nlarsch> dito
nlarsch> 
nlarsch> >      return(ret);
nlarsch> >      }

-- 
Richard Levitte   \ Spannv�gen 38, II \ [EMAIL PROTECTED]
Redakteur@Stacken  \ S-168 35  BROMMA  \ T: +46-8-26 52 47
                    \      SWEDEN       \ or +46-708-26 53 44
Procurator Odiosus Ex Infernis                -- [EMAIL PROTECTED]
Member of the OpenSSL development team: http://www.openssl.org/

Unsolicited commercial email is subject to an archival fee of $400.
See <http://www.stacken.kth.se/~levitte/mail/> for more info.
______________________________________________________________________
OpenSSL Project                                 http://www.openssl.org
Development Mailing List                       [EMAIL PROTECTED]
Automated List Manager                           [EMAIL PROTECTED]

Reply via email to