udp[6]_multi_input: Don't unlock freed inp.
ClosedPublic
Actions

Authored by jhb on Nov 27 2022, 1:25 AM.

Tags

None

Referenced Files

	F166388697: D37511.diff
	Thu, Aug 13, 7:48 AM

	Unknown Object (File)
	Tue, Aug 11, 8:57 PM

	Unknown Object (File)
	Sun, Aug 9, 2:06 PM

	Unknown Object (File)
	Fri, Aug 7, 5:53 PM

	Unknown Object (File)
	Wed, Aug 5, 8:33 AM

	Unknown Object (File)
	Wed, Aug 5, 7:45 AM

	Unknown Object (File)
	Tue, Aug 4, 3:53 AM

	Unknown Object (File)
	Sat, Jul 25, 8:44 PM

View All Files

Subscribers

imp

melifaro

Details

Reviewers

glebius
thj
ae

Group Reviewers

transport

Commits

rGd00c20882f88: udp[6]_multi_input: Don't unlock freed inp.

Summary

If udp[6]_append() returns non-zero, it is because the inp has gone
away (inpcbrele_rlocked returned 1 after running the tunnel function).

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

jhb created this revision.Nov 27 2022, 1:25 AM

Herald added a reviewer: transport. · View Herald TranscriptNov 27 2022, 1:25 AM

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

jhb requested review of this revision.Nov 27 2022, 1:25 AM

Harbormaster completed remote builds in B48511: Diff 113539.Nov 27 2022, 1:25 AM

jhb added a parent revision: D37510: ether_demux: Defer stripping the Ethernet header..Nov 27 2022, 1:25 AM

ae accepted this revision.Nov 28 2022, 6:40 AM

This revision is now accepted and ready to land.Nov 28 2022, 6:40 AM

Closed by commit rGd00c20882f88: udp[6]_multi_input: Don't unlock freed inp. (authored by jhb). · Explain WhyNov 30 2022, 10:39 PM

This revision was automatically updated to reflect the committed changes.

jhb added a commit: rGd00c20882f88: udp[6]_multi_input: Don't unlock freed inp..

Revision Contents
Changeset List

Path

Size

sys/

netinet/

udp_usrreq.c

1 line

netinet6/

udp6_usrreq.c

1 line

Diff 113723

View Options

sys/netinet/udp_usrreq.c