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)
Sun, Jan 12, 5:26 PM
Unknown Object (File)
Dec 9 2024, 3:41 PM
Unknown Object (File)
Dec 2 2024, 1:06 AM
Unknown Object (File)
Dec 2 2024, 1:04 AM
Unknown Object (File)
Nov 16 2024, 3:06 AM
Unknown Object (File)
Nov 15 2024, 8:27 PM
Unknown Object (File)
Nov 14 2024, 1:57 PM
Unknown Object (File)
Nov 14 2024, 12:12 AM
View All 25 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...