Differential D35714

tcp: remove a condition in tcp_usr_detach() that never happens
ClosedPublic
Actions

Authored by glebius on Jul 4 2022, 7:51 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Feb 23, 3:35 PM

	Unknown Object (File)
	Mon, Feb 17, 9:56 PM

	Unknown Object (File)
	Sun, Feb 16, 7:42 PM

	Unknown Object (File)
	Sat, Feb 8, 7:48 AM

	Unknown Object (File)
	Jan 21 2025, 12:56 AM

	Unknown Object (File)
	Dec 21 2024, 11:42 PM

	Unknown Object (File)
	Oct 16 2024, 8:48 PM

	Unknown Object (File)
	Sep 17 2024, 2:21 AM

View All 58 Files

Subscribers

Details

Reviewers

rrs
tuexen

Group Reviewers

Commits

rG1b91978f6375: tcp: remove a condition in tcp_usr_detach() that never happens

Summary

The comment from Robert Watson doubts that this condition ever happens.
Our analysis confirm that. Also, we found that if you manage to create
such a connection with help of some other bug, then after the "second
case" code is execute, the kernel will panic in other parts of the stack.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

glebius created this revision.Jul 4 2022, 7:51 PM

Herald added 1 blocking reviewer(s): transport. · View Herald TranscriptJul 4 2022, 7:51 PM

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

glebius requested review of this revision.Jul 4 2022, 7:51 PM

Harbormaster completed remote builds in B46262: Diff 107766.Jul 4 2022, 7:51 PM

tuexen accepted this revision.Jul 4 2022, 8:47 PM

This revision is now accepted and ready to land.Jul 4 2022, 8:47 PM

Closed by commit rG1b91978f6375: tcp: remove a condition in tcp_usr_detach() that never happens (authored by glebius). · Explain WhyJul 7 2022, 4:11 AM

This revision was automatically updated to reflect the committed changes.

glebius added a commit: rG1b91978f6375: tcp: remove a condition in tcp_usr_detach() that never happens.

Revision Contents
Changeset List

Path

Size

sys/

netinet/

24 lines

Diff 107870

sys/netinet/tcp_usrreq.c

Loading...