Author: glebius
Date: Tue Dec 23 12:26:35 2014
New Revision: 276136
URL: https://svnweb.freebsd.org/changeset/base/276136

Log:
  m_clrprotoflags() is designed to cleanup M_PROTO flags when mbufs cross
  protocol and facility boundaries.  However, now it cleans up only the
  first mbuf, while intent clearly means all mbufs in a possible chain.
  
  Differential Revision:        https://reviews.freebsd.org/D1352
  Reviewed by:          rpaulo, gnn, adrian
  Sponsored by:         Nginx, Inc.

Modified:
  head/sys/sys/mbuf.h

Modified: head/sys/sys/mbuf.h
==============================================================================
--- head/sys/sys/mbuf.h Tue Dec 23 12:13:31 2014        (r276135)
+++ head/sys/sys/mbuf.h Tue Dec 23 12:26:35 2014        (r276136)
@@ -757,7 +757,10 @@ static __inline void
 m_clrprotoflags(struct mbuf *m)
 {
 
-       m->m_flags &= ~M_PROTOFLAGS;
+       while (m) {
+               m->m_flags &= ~M_PROTOFLAGS;
+               m = m->m_next;
+       }
 }
 
 static __inline struct mbuf *
_______________________________________________
svn-src-head@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"

Reply via email to