cvs commit: src/sys/kern kern_descrip.c

Roman Divacky rdivacky at FreeBSD.org
Fri Apr 25 13:05:31 UTC 2008


On Thu, Apr 24, 2008 at 10:08:36PM +0000, Kris Kennaway wrote:
> kris        2008-04-24 22:08:36 UTC
> 
>   FreeBSD src repository
> 
>   Modified files:
>     sys/kern             kern_descrip.c 
>   Log:
>   fdhold can return NULL, so add the one remaining missing check for this
>   condition.

        fdp = fdhold(p);
+       if (fdp == NULL)
+               return (0);
        PROC_UNLOCK(p);

I think you leak the PROC lock here...


More information about the cvs-all mailing list