Page MenuHomeFreeBSD
Differential D32975

tcp: Fix a locking issue
ClosedPublic
Actions

Authored by tuexen on Nov 12 2021, 8:25 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Mar 30, 1:39 AM
Unknown Object (File)
Jan 13 2024, 11:19 PM
Unknown Object (File)
Dec 22 2023, 10:00 PM
Unknown Object (File)
Dec 15 2023, 10:35 PM
Unknown Object (File)
Dec 12 2023, 7:56 AM
Unknown Object (File)
Sep 11 2023, 3:28 AM
Unknown Object (File)
Jun 20 2023, 5:54 AM
Unknown Object (File)
May 25 2023, 7:04 PM
View All 10 Files
Subscribers
imp
melifaro

Details

Reviewers
rrs
markj
Group Reviewers
transport
Commits
rGdf07bfda67ad: tcp: Fix a locking issue
Summary

INP_WLOCK_RECHECK_CLEANUP() and INP_WLOCK_RECHECK() might return from the function, which would leave a lock held.

This fixes https://syzkaller.appspot.com/bug?id=72b2acf2475d24006a8f1a63f5a10224aeee1c8d

Diff Detail

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

Event Timeline

tuexen created this revision.Nov 12 2021, 8:25 PM
Herald added a reviewer: transport. · View Herald TranscriptNov 12 2021, 8:25 PM
Herald added subscribers: melifaro, imp. · View Herald Transcript
tuexen requested review of this revision.Nov 12 2021, 8:25 PM
markj accepted this revision.Nov 12 2021, 8:30 PM

Thanks.

sys/netinet/tcp_usrreq.c
2080

You don't need the ptr != NULL check, strictly speaking.

markj mentioned this in D32973: tcp: Fix a lock leak in tcp_congestion().Nov 12 2021, 8:30 PM
This revision was not accepted when it landed; it landed in state Needs Review.Nov 12 2021, 9:14 PM
Closed by commit rGdf07bfda67ad: tcp: Fix a locking issue (authored by tuexen). · Explain Why
This revision was automatically updated to reflect the committed changes.
tuexen added a commit: rGdf07bfda67ad: tcp: Fix a locking issue.

Revision Contents
Changeset List

PathSize
sys/
netinet/
13 lines

Diff 98447

View Options

sys/netinet/tcp_usrreq.c

Loading...