[prev in list] [next in list] [prev in thread] [next in thread]
List: netbsd-tech-net
Subject: Re: goto bad in ip_output()
From: itojun () itojun ! org (Jun-ichiro itojun Hagino)
Date: 2003-10-14 3:28:56
[Download RAW message or body]
> enami tsugutomo <enami@but-b.or.jp> writes:
>
> > Two `goto bad' in ip_output() near the end is doubtful.
> >
> > `if (error) goto bad' after the ip_fragment() just leads mbuf leak if
> > it returns ENOBUFS. the for() loop after ip_fragment() can also
> > handle error case well. And the leak actually hangs NFS.
> >
> > The `goto bad' in the if-clasue just above ip_fragment() call is
> > doubtful. Why cleanups aren't necessary?
>
> The latter was ok, I missed the `goto done;'
will fix it.
itojun
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic