Page MenuHomeFreeBSD
Differential D10073

Fix memory leak in memcontrol
AcceptedPublic
Actions

Authored by trix_juniper.net on Mar 21 2017, 1:16 PM.
Tags
Referenced Files
Unknown Object (File)
Sun, Jul 19, 3:21 AM
Unknown Object (File)
Tue, Jul 7, 9:25 PM
Unknown Object (File)
Tue, Jul 7, 2:07 AM
Unknown Object (File)
Sat, Jul 4, 10:13 AM
Unknown Object (File)
Jun 21 2026, 12:12 AM
Unknown Object (File)
Jun 20 2026, 3:54 AM
Unknown Object (File)
Jun 3 2026, 11:10 AM
Unknown Object (File)
May 17 2026, 6:30 AM
View All 95 Files
Subscribers
ed
imp

Details

Reviewers
eadler
Summary

memcontrol.c clearfunc
Free 'owner' after use

Test Plan

Use clang's static analyzer, scan-build, to find the problem and later to show resolution.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

trix_juniper.net created this revision.Mar 21 2017, 1:16 PM
Herald added a subscriber: imp. · View Herald TranscriptMar 21 2017, 1:16 PM
ed accepted this revision.Mar 21 2017, 2:32 PM
ed added inline comments.
usr.sbin/memcontrol/memcontrol.c
280

Technically speaking you should also put a free() call above the strdup()... :-)

This revision is now accepted and ready to land.Mar 21 2017, 2:32 PM
ed removed a reviewer: ed.Nov 21 2017, 9:43 PM
This revision now requires review to proceed.Nov 21 2017, 9:43 PM
eadler accepted this revision.Apr 21 2018, 4:55 PM
This revision is now accepted and ready to land.Apr 21 2018, 4:55 PM
eadler added a subscriber: ed.Apr 21 2018, 4:55 PM

obv address @ed's comments too

imp added a project: srcmgr.Aug 10 2025, 1:51 AM

Revision Contents
Changeset List

PathSize
usr.sbin/
memcontrol/
1 line

Diff 26492

View Options

usr.sbin/memcontrol/memcontrol.c

Loading...