Page MenuHomeFreeBSD
Differential D43855

tcp: partially undo D43792
ClosedPublic
Actions

Authored by rscheff on Feb 12 2024, 8:09 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, May 17, 2:37 PM
Unknown Object (File)
Fri, May 17, 11:59 AM
Unknown Object (File)
Fri, May 17, 10:56 AM
Unknown Object (File)
Mon, May 13, 12:39 PM
Unknown Object (File)
Mon, May 13, 10:00 AM
Unknown Object (File)
Fri, May 10, 4:30 AM
Unknown Object (File)
Thu, May 9, 8:01 AM
Unknown Object (File)
Thu, May 9, 1:52 AM
View All 16 Files
Subscribers
grahamperrin
imp
melifaro

Details

Reviewers
tuexen
rrs
glebius
cc
cy
Group Reviewers
transport
Commits
rG57e27ff07aff: tcp: partially undo D43792
Summary

At the destruction of the tcpcb, no timers are supposed to
be running. However, it turns out that stopping them in the
close() / shutdown() call does not have the desired effect
under all circumstances.

This partially reverts 62d47d73b7eb to reduce the nuisance
caused.

Reported-by: syzbot+9a9aa434a14a2b35c3ba@syzkaller.appspotmail.com
Reported-by: syzbot+e82856782410e895bae7@syzkaller.appspotmail.com
PR: 277009

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

rscheff created this revision.Feb 12 2024, 8:09 PM
Herald added a reviewer: transport. · View Herald TranscriptFeb 12 2024, 8:09 PM
Herald added subscribers: melifaro, imp. · View Herald Transcript
rscheff requested review of this revision.Feb 12 2024, 8:09 PM
Harbormaster completed remote builds in B55971: Diff 134224.Feb 12 2024, 8:10 PM
rscheff mentioned this in D43825: tcp: stop all timers in tcp_usr_detach().Feb 12 2024, 8:16 PM
rscheff edited the summary of this revision. (Show Details)
glebius accepted this revision.Feb 12 2024, 8:33 PM

I'm fine with the change if we all agree it is a temporary measure.

tuexen accepted this revision.Feb 12 2024, 9:36 PM
This revision is now accepted and ready to land.Feb 12 2024, 9:36 PM
Closed by commit rG57e27ff07aff: tcp: partially undo D43792 (authored by rscheff). · Explain WhyFeb 12 2024, 9:41 PM
This revision was automatically updated to reflect the committed changes.
rscheff added a commit: rG57e27ff07aff: tcp: partially undo D43792.
grahamperrin added a subscriber: grahamperrin.Feb 15 2024, 11:14 PM
glebius added a reverting change: rGe34ea0196f44: tcp: clear all TCP timers in tcp_timer_stop() when in callout.Mar 18 2024, 8:57 PM

Revision Contents
Changeset List

PathSize
sys/
netinet/
3 lines

Diff 134230

View Options

sys/netinet/tcp_subr.c

Loading...