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, May 6, 4:44 AM
Unknown Object (File)
Mon, May 4, 8:57 AM
Unknown Object (File)
Thu, Apr 30, 1:34 AM
Unknown Object (File)
Tue, Apr 28, 9:31 AM
Unknown Object (File)
Mar 5 2026, 2:14 PM
Unknown Object (File)
Feb 17 2026, 4:25 PM
Unknown Object (File)
Nov 5 2025, 9:34 PM
Unknown Object (File)
Nov 4 2025, 2:43 AM
View All 54 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...