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)
Wed, Oct 8, 9:47 AM
Unknown Object (File)
Sun, Sep 28, 5:03 PM
Unknown Object (File)
Mon, Sep 22, 1:11 AM
Unknown Object (File)
Sep 10 2025, 1:03 PM
Unknown Object (File)
Aug 13 2025, 8:18 PM
Unknown Object (File)
Jul 16 2025, 1:39 PM
Unknown Object (File)
Jun 19 2025, 3:15 PM
Unknown Object (File)
Jun 17 2025, 2:34 AM
View All 42 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...