Page MenuHomeFreeBSD
Differential D19254

Avoid dereferencing a NULL pointer when TCP_REASS_LOGGING is enabled
ClosedPublic
Actions

Authored by tuexen on Feb 19 2019, 10:14 PM.
Tags
None
Referenced Files
F154375054: D19254.id54094.diff
Tue, Apr 28, 5:05 AM
Unknown Object (File)
Mon, Apr 27, 4:19 PM
Unknown Object (File)
Mon, Apr 27, 11:30 AM
Unknown Object (File)
Mon, Apr 27, 5:17 AM
Unknown Object (File)
Sun, Apr 26, 2:43 PM
Unknown Object (File)
Fri, Apr 24, 7:11 AM
Unknown Object (File)
Sat, Apr 18, 5:45 AM
Unknown Object (File)
Tue, Apr 14, 6:39 AM
View All Files
Subscribers
imp

Details

Reviewers
bz
rrs
Group Reviewers
transport
Commits
rS344516: MFC r344428:
rS344428: This patch addresses an issue brought up by bz@ in D18968:
Summary

This patch addresses an issue brought up by bz@ in D18968: When TCP_REASS_LOGGING is defined (which doesn't compile right now), a NULL pointer dereference would happen, if user data was received during the TCP handshake and BB logging is used.

A KASSERT is also added to detect tcp_reass() calls with illegal parameter combinations.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 22585

Event Timeline

tuexen created this revision.Feb 19 2019, 10:14 PM
Herald added a reviewer: transport. · View Herald TranscriptFeb 19 2019, 10:14 PM
Herald added a subscriber: imp. · View Herald Transcript
Harbormaster completed remote builds in B22585: Diff 54094.Feb 19 2019, 10:14 PM
tuexen retitled this revision from Avoid dereferencing a NULL point when TCP_REASS_LOGGING is enabled to Avoid dereferencing a NULL pointer when TCP_REASS_LOGGING is enabled.Feb 20 2019, 6:05 PM
bz added inline comments.Feb 20 2019, 6:30 PM
sys/netinet/tcp_reass.c
547

Please add %p and others to the printf so people will be able to have pointers to the tp, th, ..., m for debugging in case this fires.
Will also help them to find out which illegal combination it was.

tuexen updated this revision to Diff 54147.Feb 20 2019, 7:23 PM

Address bz@'s comment.

Harbormaster completed remote builds in B22625: Diff 54147.Feb 20 2019, 7:23 PM
tuexen marked an inline comment as done.Feb 20 2019, 7:26 PM
tuexen added inline comments.
sys/netinet/tcp_reass.c
547

Good suggestion.Diff updated.

rrs accepted this revision.Feb 20 2019, 8:31 PM
This revision is now accepted and ready to land.Feb 20 2019, 8:31 PM
tuexen marked an inline comment as done.Feb 21 2019, 1:03 AM
Closed by commit rS344428: This patch addresses an issue brought up by bz@ in D18968: (authored by tuexen). · Explain WhyFeb 21 2019, 9:34 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
sys/
netinet/
20 lines

Diff 54094

View Options

sys/netinet/tcp_reass.c

Loading...