if ((a->method == NULL) || (a->method->destroy == NULL)) return(1);

 136         a->method->destroy(a);

 137         OPENSSL_free(a);

 138         return(1);

if haven't method destroy memory for a is never free.
should write like this
 135         if ((a->method == NULL) || (a->method->destroy == NULL))
a->method->destroy(a);

 137         OPENSSL_free(a);

 138         return(1);


On Tue, Jul 8, 2014 at 1:42 AM, The default queue via RT <[email protected]>
wrote:

>
> Greetings,
>
> This message has been automatically generated in response to the
> creation of a trouble ticket regarding:
>         "Memory leak bug",
> a summary of which appears below.
>
> There is no need to reply to this message right now.  Your ticket has been
> assigned an ID of [openssl.org #3439].
>
> Please include the string:
>
>          [openssl.org #3439]
>
> in the subject line of all future correspondence about this issue. To do
> so,
> you may reply to this message.
>
>                         Thank you,
>                         [email protected]
>
> -------------------------------------------------------------------------
> In [openssl.git]
> <
> http://git.openssl.org/gitweb/?p=openssl.git;a=tree;hb=b73ac027357da29d9e393f24cd224999c94028d1
> >
>  / crypto
> <
> http://git.openssl.org/gitweb/?p=openssl.git;a=tree;f=crypto;hb=b73ac027357da29d9e393f24cd224999c94028d1
> >
>  / bio
> <
> http://git.openssl.org/gitweb/?p=openssl.git;a=tree;f=crypto/bio;hb=b73ac027357da29d9e393f24cd224999c94028d1
> >
>  / bio_lib.c
> <
> http://git.openssl.org/gitweb/?p=openssl.git;a=blob_plain;f=crypto/bio/bio_lib.c;hb=b73ac027357da29d9e393f24cd224999c94028d1
> >
> in int BIO_free(BIO *a)
>
> 135
> <
> http://git.openssl.org/gitweb/?p=openssl.git;a=blob;f=crypto/bio/bio_lib.c;h=9c9646afa8a22c3a6a208877c76e7f92d8458f31;hb=b73ac027357da29d9e393f24cd224999c94028d1#l135
> >
>         if ((a->method == NULL) || (a->method->destroy == NULL)) return(1);
> 136
> <
> http://git.openssl.org/gitweb/?p=openssl.git;a=blob;f=crypto/bio/bio_lib.c;h=9c9646afa8a22c3a6a208877c76e7f92d8458f31;hb=b73ac027357da29d9e393f24cd224999c94028d1#l136
> >
>         a->method->destroy(a);
> 137
> <
> http://git.openssl.org/gitweb/?p=openssl.git;a=blob;f=crypto/bio/bio_lib.c;h=9c9646afa8a22c3a6a208877c76e7f92d8458f31;hb=b73ac027357da29d9e393f24cd224999c94028d1#l137
> >
>         OPENSSL_free(a);
> 138
> <
> http://git.openssl.org/gitweb/?p=openssl.git;a=blob;f=crypto/bio/bio_lib.c;h=9c9646afa8a22c3a6a208877c76e7f92d8458f31;hb=b73ac027357da29d9e393f24cd224999c94028d1#l138
> >
>         return(1);
> if haven't method destroy memory for a is never free.
> should write like this
> 135
> <
> http://git.openssl.org/gitweb/?p=openssl.git;a=blob;f=crypto/bio/bio_lib.c;h=9c9646afa8a22c3a6a208877c76e7f92d8458f31;hb=b73ac027357da29d9e393f24cd224999c94028d1#l135
> >
>         if ((a->method == NULL) || (a->method->destroy == NULL))
> a->method->destroy(a);
> 137
> <
> http://git.openssl.org/gitweb/?p=openssl.git;a=blob;f=crypto/bio/bio_lib.c;h=9c9646afa8a22c3a6a208877c76e7f92d8458f31;hb=b73ac027357da29d9e393f24cd224999c94028d1#l137
> >
>         OPENSSL_free(a);
> 138
> <
> http://git.openssl.org/gitweb/?p=openssl.git;a=blob;f=crypto/bio/bio_lib.c;h=9c9646afa8a22c3a6a208877c76e7f92d8458f31;hb=b73ac027357da29d9e393f24cd224999c94028d1#l138
> >
>         return(1);
>
> Best regards,
> Korzhetsky Yury
>
>

______________________________________________________________________
OpenSSL Project                                 http://www.openssl.org
Development Mailing List                       [email protected]
Automated List Manager                           [email protected]

Reply via email to