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)
Thu, Oct 15, 6:49 PM
Unknown Object (File)
Thu, Oct 15, 2:15 PM
Unknown Object (File)
Sep 7 2026, 7:36 PM
Unknown Object (File)
Sep 6 2026, 11:39 PM
Unknown Object (File)
Sep 6 2026, 3:09 AM
Unknown Object (File)
Sep 5 2026, 2:03 AM
Unknown Object (File)
Sep 4 2026, 7:17 AM
Unknown Object (File)
Aug 29 2026, 12:04 PM
View All 72 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...