Hello!

On Thu, Mar 13, 2003 at 05:15:03PM +0100, Dave Jones wrote:

>  >    Seems this is right list/person to post this.
>  >    I am looking at ./drivers/char/drm/drm_drv.h::drm_init() (latest 2.4), and
>  >    it seems there is a memleak on error exit path.
>  >    If DRM(stub_register)(DRIVER_NAME, &DRM(fops),dev) fails,
>  >    DRM(device) and DRM(minor) memory areas are not freed before exit.
>  >    Is this looking real?
> Looks like a leak, smells like a leak, afaics, is a leak.
> The exit paths further down also look fishy, as takedown() doesn't
> free that stuff, so its left dangling.

Yes, the script complained about other exit paths too, but I have not investigated
whenever those other cleanup functions free the initially allocated memory or not.
The code does not look pretty to me, I am only 50% through list of potential leaks,
so I do not want to spends lots of time on specific things right now, my current
goal is to make number of false positives down.

Bye,
    Oleg


-------------------------------------------------------
This SF.net email is sponsored by:Crypto Challenge is now open! 
Get cracking and register here for some mind boggling fun and 
the chance of winning an Apple iPod:
http://ads.sourceforge.net/cgi-bin/redirect.pl?thaw0031en
_______________________________________________
Dri-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/dri-devel

Reply via email to