Re: PR/53998 CVS commit: src/sys/kern

[Christos Zoulas <christos%zoulas.com@localhost>]

The following reply was made to PR kern/53998; it has been noted by GNATS.

From: Christos Zoulas <christos%zoulas.com@localhost>
To: "gnats-bugs%netbsd.org@localhost" <gnats-bugs%NetBSD.org@localhost>
Cc: kern-bug-people%netbsd.org@localhost,
 gnats-admin%netbsd.org@localhost,
 netbsd-bugs%netbsd.org@localhost,
 joel.bertrand%systella.fr@localhost
Subject: Re: PR/53998 CVS commit: src/sys/kern
Date: Thu, 21 Feb 2019 18:58:55 -0500

 > 
 > 	Christos,
 > 
 > 	Thanks a lot for this patch, but I'm not really sure it will fix this
 > issue. Of course, with your patch, sem_init() will return ENOSPC instead
 > of -1. but I'm not understand how a memory allocation error can occur in
 > my code. I have verified that my test program doesn't contain memory
 > leak and, when sem_init() fails, system has more than 10GB free.
 > 
 > 	Of course, I create a semahore with sem_init() in each thread but I
 > destroy this semaphore also with sem_destroy() (returns 0) before
 > pthread_exit().
 
 Yes, I understand, I am documenting that this also fails when the
 max number of semaphors is exceeded. So perhaps we have a leak
 somewhere. Can you share your test program?
 
 christos