tcp: Blackbox logging and tcp accounting together can cause a crash.
ClosedPublic
Actions

Authored by rrs on Apr 17 2023, 7:06 PM.

Tags

None

Referenced Files

	F144477190: D39622.id120497.diff
	Sun, Feb 8, 3:44 PM

	F144404479: D39622.id120498.diff
	Sun, Feb 8, 6:36 AM

	Unknown Object (File)
	Sat, Jan 31, 8:14 AM

	Unknown Object (File)
	Thu, Jan 29, 11:23 PM

	Unknown Object (File)
	Wed, Jan 28, 11:47 PM

	Unknown Object (File)
	Wed, Jan 28, 2:00 PM

	Unknown Object (File)
	Wed, Jan 28, 9:02 AM

	Unknown Object (File)
	Wed, Jan 28, 7:58 AM

Subscribers

Details

Reviewers

tuexen

Group Reviewers

transport

Commits

rG37229fed38c6: tcp: Blackbox logging and tcp accounting together can cause a crash.

Summary

If you currently turn BB logging on and in combination have TCP Accounting on we can get a
crash where we have no NULL check and we run out of memory. Also lets make sure we
don't do a divide by 0 in calculating any BB ratios.

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

rrs created this revision.Apr 17 2023, 7:06 PM

Herald added 1 blocking reviewer(s): transport. · View Herald TranscriptApr 17 2023, 7:06 PM

Herald added subscribers: glebius, melifaro. · View Herald Transcript

rrs requested review of this revision.Apr 17 2023, 7:06 PM

Opps lets get the sets of the log union in the right place.

tuexen accepted this revision.Apr 17 2023, 8:33 PM

This revision is now accepted and ready to land.Apr 17 2023, 8:33 PM

Closed by commit rG37229fed38c6: tcp: Blackbox logging and tcp accounting together can cause a crash. (authored by rrs). · Explain WhyApr 18 2023, 12:40 PM

This revision was automatically updated to reflect the committed changes.

rrs added a commit: rG37229fed38c6: tcp: Blackbox logging and tcp accounting together can cause a crash..

Herald added a subscriber: imp. · View Herald TranscriptApr 18 2023, 12:40 PM

Revision Contents
Changeset List

Path

Size

sys/

netinet/

tcp_log_buf.c

38 lines

Diff 120498

View Options

tcp: Blackbox logging and tcp accounting together can cause a crash.ClosedPublicActions