Re: PATCH : fixed memory leak in saslauthd/auth_krb5.c


Subject: Re: PATCH : fixed memory leak in saslauthd/auth_krb5.c
From: Rob Siemborski (rjs3 at andrew dot cmu dot edu)
Date: Tue May 06 2003 - 15:05:56 EDT


I'm pretty sure this should actually be

    if (keyblock) {
        krb5_free_keyblock(context, keyblock);
    }

-Rob

On Fri, 2 May 2003, Etienne Goyer wrote:

> Hi,
>
> I think I found a memory leak in saslauthd/auth_krb5.c. Here is the
> patch to my fix.
>
> --- auth_krb5.c.orig Fri May 2 16:26:56 2003
> +++ auth_krb5.c Fri May 2 16:36:14 2003
> @@ -221,6 +221,7 @@
> }
>
> if (keyblock) {
> + free(keyblock->contents);
> free(keyblock);
> }
>
> --
> Etienne Goyer Linux Québec Technologies Inc.
> http://www.LinuxQuebec.com etienne dot goyer at linuxquebec dot com
> PGP Pub Key: http://www.LinuxQuebec.com/pubkeys/eg.key
> Fingerprint: F569 0394 098A FC70 B572 5D20 3129 3D86 8FD5 C853
>
>

-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-
Rob Siemborski * Andrew Systems Group * Cyert Hall 207 * 412-268-7456
Research Systems Programmer * /usr/contributed Gatekeeper







Hosted Email Solutions

Invaluement Anti-Spam DNSBLs



Powered By FreeBSD   Powered By FreeBSD