cvs commit: src/sys/kern kern_descrip.c
Dmitry Sivachenko
demon at freebsd.org
Thu Jun 19 12:12:14 PDT 2003
On Thu, Jun 19, 2003 at 11:22:13AM -0700, John-Mark Gurney wrote:
> Mike Silbersack wrote this message on Thu, Jun 19, 2003 at 14:03 -0500:
> >
> > On Thu, 19 Jun 2003, Dmitry Sivachenko wrote:
> >
> > > > Oh, btw; while running my fd hog program as root, cron decided to core.
> > > > Someone should take a look at it and figure out where it doesn't check the
> > > > return value of something which allocates a fd (or socket?)
> > > >
> > >
> > > bin/49096 ?
> >
> > That looks like it might be it, I'll give it a try and commit it if solves
> > the problem for me.
>
> Actually, that patch should be committed even if it doesn't solve the
> problem. The if (in == NULL) is useless since we already tried to
> read a char from it.
>
I tested it before submitting trying to reproduce symptoms desctibed in
the PR. This patch solved the problem for me.
FYI.
More information about the cvs-src
mailing list