Differential D38573

inpcb: Release the inpcb cred reference before freeing the structure
ClosedPublic
Actions

Authored by markj on Feb 13 2023, 11:17 PM.

Details

Reviewers

glebius

Group Reviewers

network

Commits

rG5fd1a67e885e: inpcb: Release the inpcb cred reference before freeing the structure

Summary

Now that the inp_cred point is accessed only when the inpcb lock is
held, we can avoid deferring a crfree() call when freeing an inpcb.

Sponsored by: Klara, Inc.
Sponsored by: Modirum MDPay

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

markj created this revision.Feb 13 2023, 11:17 PM

Herald added subscribers: melifaro, imp. · View Herald TranscriptFeb 13 2023, 11:17 PM

markj requested review of this revision.Feb 13 2023, 11:17 PM

Harbormaster completed remote builds in B49755: Diff 117158.Feb 13 2023, 11:17 PM

markj added a parent revision: D38572: inpcb: Avoid inp_cred dereferences in SMR-protected lookup.Feb 13 2023, 11:17 PM

markj added a child revision: D38574: netinet: Remove IP(V6)_BINDMULTI.

zlei added a subscriber: zlei.Feb 14 2023, 10:20 AM

dch added a subscriber: dch.Feb 16 2023, 10:35 PM

This revision was not accepted when it landed; it landed in state Needs Review.Apr 20 2023, 4:14 PM

Closed by commit rG5fd1a67e885e: inpcb: Release the inpcb cred reference before freeing the structure (authored by markj). · Explain Why

This revision was automatically updated to reflect the committed changes.

markj added a commit: rG5fd1a67e885e: inpcb: Release the inpcb cred reference before freeing the structure.

pouria awarded a token.Aug 7 2023, 7:15 PM

Revision Contents
Changeset List

Path

Size

sys/

netinet/

in_pcb.c

23 lines

Diff 120728

View Options

inpcb: Release the inpcb cred reference before freeing the structureClosedPublicActions