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

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

Tags

None

Referenced Files

	F149791092: D1352.id2812.diff
	Fri, Mar 27, 3:10 AM

	Unknown Object (File)
	Wed, Mar 25, 9:18 AM

	Unknown Object (File)
	Wed, Mar 25, 5:54 AM

	Unknown Object (File)
	Sun, Mar 22, 4:07 AM

	Unknown Object (File)
	Sat, Mar 21, 10:17 AM

	Unknown Object (File)
	Sat, Mar 7, 5:45 PM

	Unknown Object (File)
	Dec 24 2025, 11:43 PM

	Unknown Object (File)
	Dec 22 2025, 9:15 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

Lint Skipped

Unit

Tests Skipped

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

head/

sys/

mbuf.h

5 lines

Diff 2827

View Options

In m_clrprotoflags() cleanup the entire mbuf chain.ClosedPublicActions