Page MenuHomeFreeBSD
Differential D45488

radlib: fix a memory leak
ClosedPublic
Actions

Authored by ngie on Jun 4 2024, 8:03 PM.
Tags
None
Referenced Files
F133292969: D45488.id139482.diff
Fri, Oct 24, 5:00 PM
Unknown Object (File)
Mon, Oct 20, 1:52 AM
Unknown Object (File)
Sun, Oct 12, 5:23 PM
Unknown Object (File)
Fri, Oct 10, 3:37 PM
Unknown Object (File)
Sat, Oct 4, 1:34 AM
Unknown Object (File)
Fri, Oct 3, 8:34 PM
Unknown Object (File)
Fri, Oct 3, 2:54 AM
Unknown Object (File)
Thu, Oct 2, 8:42 AM
View All 86 Files
Subscribers
erj
imp
markj

Details

Reviewers
emaste
erj
markj
allanjude
Commits
rGf7d46b858a54: radlib: fix a memory leak in `is_valid_request`
rGc2dca1b5607f: radlib: fix a memory leak in `is_valid_request`
rG77c04f3eb12a: radlib: fix a memory leak in `is_valid_request`
Summary

Call HMAC_CTX_free if returning early from is_valid_request when
processing Message-Authenticator tags.

Reported by: Coverity

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 58057
Build 54945: arc lint + arc unit

Event Timeline

ngie created this revision.Jun 4 2024, 8:03 PM
Herald added a subscriber: imp. · View Herald TranscriptJun 4 2024, 8:03 PM
ngie requested review of this revision.Jun 4 2024, 8:03 PM
Harbormaster completed remote builds in B58057: Diff 139467.Jun 4 2024, 8:03 PM
ngie edited the summary of this revision. (Show Details)Jun 4 2024, 8:04 PM
ngie added a reviewer: emaste.
erj accepted this revision.Jun 4 2024, 8:07 PM
erj added a subscriber: erj.

It looks like it matches the cleanup that happens in the other return paths to me

This revision is now accepted and ready to land.Jun 4 2024, 8:07 PM
markj accepted this revision.Jun 4 2024, 8:09 PM
markj added a subscriber: markj.

Please add Fixes: 8d5c7813061d ("libradius: Fix input validation bugs") to the commit log message.

allanjude accepted this revision.Jun 4 2024, 8:12 PM
emaste accepted this revision.Jun 4 2024, 8:14 PM
ngie added a comment.Jun 5 2024, 2:40 AM
In D45488#1037640, @markj wrote:

Please add Fixes: 8d5c7813061d ("libradius: Fix input validation bugs") to the commit log message.

Done!

Closed by commit rG77c04f3eb12a: radlib: fix a memory leak in `is_valid_request` (authored by ngie). · Explain WhyJun 5 2024, 2:42 AM
This revision was automatically updated to reflect the committed changes.
ngie added a commit: rG77c04f3eb12a: radlib: fix a memory leak in `is_valid_request`.
ngie added a commit: rGc2dca1b5607f: radlib: fix a memory leak in `is_valid_request`.Sep 28 2024, 3:55 AM
ngie added a commit: rGf7d46b858a54: radlib: fix a memory leak in `is_valid_request`.Mar 26 2025, 9:16 PM

Revision Contents
Changeset List

PathSize
lib/
libradius/
4 lines

Diff 139467

View Options

lib/libradius/radlib.c

Loading...