Re: CVS commit: src/sys/kern

[Masanobu SAITOH <msaitoh%execsw.org@localhost>]

On 2015/02/09 13:27, Masanobu SAITOH wrote:
> Hi!!!!!!!!!!!!
>
> On 2015/02/08 23:46, Michael van Elst wrote:
> > Module Name:    src
> > Committed By:    mlelstv
> > Date:        Sun Feb  8 14:46:30 UTC 2015
> >
> > Modified Files:
> >     src/sys/kern: uipc_mbuf.c
> >
> > Log Message:
> > Correct m_len calculation for m_dup() with mbuf clusters.
> > Fixes kern/49650.
> >
> >
> > To generate a diff of this commit:
> > cvs rdiff -u -r1.160 -r1.161 src/sys/kern/uipc_mbuf.c
> >
> > Please note that diffs are not public domain; they are subject to the
> > copyright notices on the relevant files.
>
> It restored only one of four deletion of m_len = 0
>
> The original was rev. 1.153:
>
>      http://cvsweb.netbsd.org/bsdweb.cgi/src/sys/kern/uipc_mbuf.c.diff?r1=1.152&r2=1.153&f=h
>
> Other three were also used after MGET() or MCLGET().
> Should we restore them, too?

 Oh, sorry. I understood that "m->m_len = 0" is done in m_get().

--
-----------------------------------------------
                SAITOH Masanobu (msaitoh%execsw.org@localhost
                                 msaitoh%netbsd.org@localhost)