if_vxlan(4): Move the allocating of statistics counters below if_alloc()
ClosedPublic
Actions

Authored by zlei on Jul 14 2024, 3:07 PM.

Details

Reviewers

kp
glebius

Group Reviewers

network

Commits

rGa948a61a822f: if_vxlan(4): Move the allocating of statistics counters below if_alloc()

Summary

The MFS of the fix [1] is done but the prerequisite [2] has not been
MFCed to stable/13 branch yet. This result in another potential leak
condition in stable/13 branch. Well if_alloc(IFT_ETHER) shall not
fail, but just in case.

This is direct commit to stable/13.

e729e750806d if_vxlan(4): Plug a memory leak
4787572d0580 ifnet: make if_alloc_domain() never fail

Fixes: 12bd1341d2aa if_vxlan(4): Plug a memory leak

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

zlei created this revision.Jul 14 2024, 3:07 PM

Herald added subscribers: melifaro, ae, imp. · View Herald TranscriptJul 14 2024, 3:07 PM

zlei requested review of this revision.Jul 14 2024, 3:07 PM

kp accepted this revision as: kp.Jul 15 2024, 1:47 PM

This revision is now accepted and ready to land.Jul 15 2024, 1:47 PM

Closed by commit rGa948a61a822f: if_vxlan(4): Move the allocating of statistics counters below if_alloc() (authored by zlei). · Explain WhyJul 16 2024, 3:55 PM

This revision was automatically updated to reflect the committed changes.

zlei added a commit: rGa948a61a822f: if_vxlan(4): Move the allocating of statistics counters below if_alloc().

Revision Contents
Changeset List

Path

Size

sys/

net/

if_vxlan.c

2 lines

Diff 141010

View Options

if_vxlan(4): Move the allocating of statistics counters below if_alloc()ClosedPublicActions