Re: svn commit: r236917 - head/sys/kern

2012-06-13 Thread Pawel Jakub Dawidek
On Wed, Jun 13, 2012 at 08:09:01AM +0400, Andrey Chernov wrote: > On Wed, Jun 13, 2012 at 12:37:50PM +1000, Bruce Evans wrote: > > >> On Mon, 11 Jun 2012, Pawel Jakub Dawidek wrote: > > >>> -KASSERT(fd >= 0 && fd < fdp->fd_nfiles, > > >>> +KASSERT((unsigned int)fd < fdp->fd_nfiles,

Re: svn commit: r236917 - head/sys/kern

2012-06-12 Thread Andrey Chernov
On Wed, Jun 13, 2012 at 12:37:50PM +1000, Bruce Evans wrote: > >> On Mon, 11 Jun 2012, Pawel Jakub Dawidek wrote: > >>> -KASSERT(fd >= 0 && fd < fdp->fd_nfiles, > >>> +KASSERT((unsigned int)fd < fdp->fd_nfiles, > >>> ("file descriptor %d out of range (0, %d)", fd, > >>>

Re: svn commit: r236917 - head/sys/kern

2012-06-12 Thread Bruce Evans
On Tue, 12 Jun 2012, Pawel Jakub Dawidek wrote: On Tue, Jun 12, 2012 at 12:53:47PM +1000, Bruce Evans wrote: On Mon, 11 Jun 2012, Pawel Jakub Dawidek wrote: -KASSERT(fd >= 0 && fd < fdp->fd_nfiles, +KASSERT((unsigned int)fd < fdp->fd_nfiles, ("file descriptor %d out

Re: svn commit: r236917 - head/sys/kern

2012-06-12 Thread Pawel Jakub Dawidek
On Tue, Jun 12, 2012 at 12:53:47PM +1000, Bruce Evans wrote: > On Mon, 11 Jun 2012, Pawel Jakub Dawidek wrote: > > -KASSERT(fd >= 0 && fd < fdp->fd_nfiles, > > +KASSERT((unsigned int)fd < fdp->fd_nfiles, > > ("file descriptor %d out of range (0, %d)", fd, > > fdp->fd_nf

Re: svn commit: r236917 - head/sys/kern

2012-06-11 Thread Bruce Evans
On Mon, 11 Jun 2012, Pawel Jakub Dawidek wrote: Log: Use consistent way of checking if descriptor number is valid. MFC after: 1 month Modified: head/sys/kern/kern_descrip.c Modified: head/sys/kern/kern_descrip.c ==

svn commit: r236917 - head/sys/kern

2012-06-11 Thread Pawel Jakub Dawidek
Author: pjd Date: Mon Jun 11 20:17:20 2012 New Revision: 236917 URL: http://svn.freebsd.org/changeset/base/236917 Log: Use consistent way of checking if descriptor number is valid. MFC after:1 month Modified: head/sys/kern/kern_descrip.c Modified: head/sys/kern/kern_descrip.c ==