Date: Fri, 8 May 2009 01:54:30 GMT From: Martin Karsten <mkarsten@cs.uwaterloo.ca> To: freebsd-gnats-submit@FreeBSD.org Subject: kern/134355: cooments for m_getm2 inconsistent with behaviour Message-ID: <200905080154.n481sUWh048544@www.freebsd.org> Resent-Message-ID: <200905080200.n482087A040299@freefall.freebsd.org>
next in thread | raw e-mail | index | archive | help
>Number: 134355 >Category: kern >Synopsis: cooments for m_getm2 inconsistent with behaviour >Confidential: no >Severity: non-critical >Priority: low >Responsible: freebsd-bugs >State: open >Quarter: >Keywords: >Date-Required: >Class: sw-bug >Submitter-Id: current-users >Arrival-Date: Fri May 08 02:00:08 UTC 2009 >Closed-Date: >Last-Modified: >Originator: Martin Karsten >Release: 7.2 >Organization: University of Waterloo >Environment: FreeBSD i01.expnet 7.2-RELEASE FreeBSD 7.2-RELEASE #0: Fri May 1 08:49:13 UTC 2009 root@walker.cse.buffalo.edu:/usr/obj/usr/src/sys/GENERIC i386 >Description: It seems to me that the comments describing m_getm2 are inconsistent with the behaviour. The comment states that the function returns the top of the newly allocated chain, but it in fact returns the top of the overall chain. >How-To-Repeat: N/A >Fix: To return the top of the newly allocated chain, the last statement in line 151 would need to be changed from return (m); to return (nm); and the very last else-clause before that is not needed. >Release-Note: >Audit-Trail: >Unformatted:
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200905080154.n481sUWh048544>