Re: semctl(2) SETVAL/SETALL does not validate the semaphore value

[Mindaugas Rasiukevicius <rmind%NetBSD.org@localhost>]

Nicolas Joly <njoly%pasteur.fr@localhost> wrote:
> +                     if ((unsigned int)semval > seminfo.semvmx) {
> +                             error = ERANGE;
> +                             break;
> +                     semaptr->_sem_base[i].semval = semval;
> +                     }

I think you wanted bracket after the "break".
In general looks fine.

-- 
Best regards,
Mindaugas
www.NetBSD.org