Page MenuHomeFreeBSD
Differential D25074

ctld: Fix leaks in conf_apply().
AbandonedPublic
Actions

Authored by markj on May 30 2020, 7:05 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Sep 7, 7:36 PM
Unknown Object (File)
Sun, Sep 6, 11:39 PM
Unknown Object (File)
Sun, Sep 6, 3:09 AM
Unknown Object (File)
Sat, Sep 5, 2:03 AM
Unknown Object (File)
Fri, Sep 4, 7:17 AM
Unknown Object (File)
Sat, Aug 29, 12:04 PM
Unknown Object (File)
Sat, Aug 29, 12:00 PM
Unknown Object (File)
Thu, Aug 27, 7:56 PM
View All 70 Files
Subscribers
None

Details

Reviewers
mav
trasz
Summary

PR: 246596

Diff Detail

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 31397
Build 29021: arc lint + arc unit

Event Timeline

markj requested review of this revision.May 30 2020, 7:05 PM
markj created this revision.
Harbormaster completed remote builds in B31397: Diff 72454.May 30 2020, 7:05 PM
markj added reviewers: mav, trasz.May 30 2020, 7:05 PM
mav accepted this revision.May 31 2020, 2:04 PM

It should be OK if desided, but I don't think there is a leak, since the whole old configuration is freed after application complete.

This revision is now accepted and ready to land.May 31 2020, 2:04 PM
markj abandoned this revision.May 31 2020, 6:55 PM
In D25074#552314, @mav wrote:

It should be OK if desided, but I don't think there is a leak, since the whole old configuration is freed after application complete.

I see now, thank you. I will just leave the code alone then.

Revision Contents
Changeset List

PathSize
usr.sbin/
ctld/
2 lines

Diff 72454

View Options

usr.sbin/ctld/ctld.c

Loading...