Page MenuHomeFreeBSD
Differential D31194

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, Apr 21, 2:04 PM
Unknown Object (File)
Tue, Apr 14, 12:22 AM
Unknown Object (File)
Sat, Apr 11, 11:40 PM
Unknown Object (File)
Fri, Apr 10, 6:01 AM
Unknown Object (File)
Mon, Apr 6, 8:57 AM
Unknown Object (File)
Mon, Apr 6, 5:21 AM
Unknown Object (File)
Sun, Apr 5, 10:40 AM
Unknown Object (File)
Sun, Mar 29, 4:01 AM
View All 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

Revision Contents
Changeset List

PathSize
sys/
netinet/
tcp_stacks/
2 lines

Diff 92287

View Options

sys/netinet/tcp_stacks/rack_bbr_common.c

Loading...