Page MenuHomeFreeBSD

Remove an extra if_ref().
ClosedPublic

Authored by jhb on Oct 28 2020, 11:53 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Oct 15, 9:01 AM
Unknown Object (File)
Wed, Oct 15, 7:03 AM
Unknown Object (File)
Sun, Oct 12, 2:12 AM
Unknown Object (File)
Wed, Oct 1, 1:56 AM
Unknown Object (File)
Aug 27 2025, 11:34 PM
Unknown Object (File)
Aug 26 2025, 2:42 AM
Unknown Object (File)
Aug 2 2025, 3:04 PM
Unknown Object (File)
Jul 5 2025, 1:57 PM

Details

Reviewers
gallatin
rrs
Group Reviewers
transport
Commits
rS367147: Remove an extra if_ref().
Summary

In r348254, if_snd_tag_alloc() routines were changed to bump the ifp
refcount via m_snd_tag_init(). This function wasn't in the tree at
the time and wasn't updated for the new semantics, so was still doing
a separate bump after if_snd_tag_alloc() returned.

Test Plan
  • tested by Drew

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

jhb requested review of this revision.Oct 28 2020, 11:53 PM
This revision was not accepted when it landed; it landed in state Needs Review.Oct 29 2020, 10:17 PM
Closed by commit rS367147: Remove an extra if_ref(). (authored by jhb). · Explain Why
This revision was automatically updated to reflect the committed changes.

Sorry for the delayed review. This fix looks correct to me.