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)
	Sun, Apr 6, 5:04 AM

	Unknown Object (File)
	Sat, Mar 22, 9:18 AM

	Unknown Object (File)
	Mar 14 2025, 11:09 PM

	Unknown Object (File)
	Feb 9 2025, 10:23 PM

	Unknown Object (File)
	Feb 9 2025, 11:22 AM

	Unknown Object (File)
	Feb 3 2025, 2:48 PM

	Unknown Object (File)
	Jan 26 2025, 8:50 PM

	Unknown Object (File)
	Dec 14 2024, 6:36 AM

View All 60 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