In m_clrprotoflags() cleanup the entire mbuf chain.
ClosedPublic
Actions

Authored by glebius on Dec 22 2014, 3:50 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Oct 11, 10:50 PM

	Unknown Object (File)
	Fri, Oct 2, 7:45 PM

	Unknown Object (File)
	Thu, Oct 1, 5:42 PM

	Unknown Object (File)
	Thu, Oct 1, 4:00 AM

	Unknown Object (File)
	Thu, Oct 1, 3:33 AM

	Unknown Object (File)
	Wed, Sep 30, 4:09 PM

	Unknown Object (File)
	Wed, Sep 30, 4:01 PM

	Unknown Object (File)
	Wed, Sep 30, 2:39 PM

Subscribers

Details

Reviewers

rpaulo
gnn
adrian

Group Reviewers

network

Commits

rS276136: m_clrprotoflags() is designed to cleanup M_PROTO flags when mbufs cross

Summary

The function 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.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

glebius updated this revision to Diff 2812.Dec 22 2014, 3:50 PM

glebius retitled this revision from to In m_clrprotoflags() cleanup the entire mbuf chain..

glebius updated this object.

glebius added a reviewer: network.

In the commit log, I wouldn't use the pronoun "the" but instead you could make it clear you're talking about m_clrprotoflags().

This revision is now accepted and ready to land.Dec 22 2014, 6:11 PM

Looks good to me.

Yup, looks good to me too.

Closed by commit rS276136 (authored by @glebius).

Revision Contents
Changeset List

Path

Size

sys/

mbuf.h

5 lines

Diff 2812

View Options

In m_clrprotoflags() cleanup the entire mbuf chain.ClosedPublicActions