tcp: When rack or bbr get a pullup failure in the common code, don't free the NULL mbuf.
ClosedPublic
Actions

Authored by rrs on Jul 15 2021, 3:09 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, Nov 25, 11:52 PM

	Unknown Object (File)
	Tue, Nov 25, 4:26 PM

	Unknown Object (File)
	Mon, Nov 24, 4:39 AM

	Unknown Object (File)
	Sat, Nov 22, 2:59 PM

	Unknown Object (File)
	Nov 15 2025, 11:03 PM

	Unknown Object (File)
	Oct 26 2025, 3:19 AM

	Unknown Object (File)
	Oct 24 2025, 1:47 PM

	Unknown Object (File)
	Oct 13 2025, 12:53 PM

View All 96 Files

Subscribers

melifaro

Details

Reviewers

tuexen

Group Reviewers

transport

Commits

rGdb4d2d7222ee: tcp: When rack or bbr get a pullup failure in the common code, don't free the…

Summary

There is a bug in the error path where rack_bbr_common does a m_pullup() and the pullup fails.
There is a stray mfree(m) after m is set to NULL. This is not a good idea :-)

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

rrs created this revision.Jul 15 2021, 3:09 PM

Herald added 1 blocking reviewer(s): transport. · View Herald TranscriptJul 15 2021, 3:09 PM

Herald added a subscriber: melifaro. · View Herald Transcript

rrs requested review of this revision.Jul 15 2021, 3:09 PM

tuexen accepted this revision.Jul 16 2021, 10:43 AM

This revision is now accepted and ready to land.Jul 16 2021, 10:43 AM

Closed by commit rGdb4d2d7222ee: tcp: When rack or bbr get a pullup failure in the common code, don't free the… (authored by rrs). · Explain WhyJul 16 2021, 6:01 PM

This revision was automatically updated to reflect the committed changes.

rrs added a commit: rGdb4d2d7222ee: tcp: When rack or bbr get a pullup failure in the common code, don't free the….

Committed

Revision Contents
Changeset List

Path

Size

sys/

netinet/

tcp_stacks/

rack_bbr_common.c

2 lines

Diff 92287

View Options

sys/netinet/tcp_stacks/rack_bbr_common.c

tcp: When rack or bbr get a pullup failure in the common code, don't free the NULL mbuf.ClosedPublicActions