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)
Fri, Sep 4, 6:40 PM
Unknown Object (File)
Fri, Sep 4, 6:22 PM
Unknown Object (File)
Wed, Sep 2, 12:40 AM
Unknown Object (File)
Mon, Aug 31, 12:36 AM
Unknown Object (File)
Sat, Aug 22, 1:17 PM
Unknown Object (File)
Aug 7 2026, 5:20 PM
Unknown Object (File)
Aug 1 2026, 6:06 AM
Unknown Object (File)
Jul 30 2026, 8:51 PM
View All 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...