Page MenuHomeFreeBSD
Differential D9676

Fix memory leaks in route
ClosedPublic
Actions

Authored by trix_juniper.net on Feb 19 2017, 5:52 PM.
Tags
None
Referenced Files
Unknown Object (File)
Nov 26 2024, 8:18 AM
Unknown Object (File)
Nov 24 2024, 6:01 PM
Unknown Object (File)
Nov 23 2024, 5:48 PM
Unknown Object (File)
Nov 23 2024, 3:58 AM
Unknown Object (File)
Nov 14 2024, 3:30 PM
Unknown Object (File)
Nov 7 2024, 12:11 AM
Unknown Object (File)
Nov 5 2024, 1:10 PM
Unknown Object (File)
Oct 17 2024, 5:29 AM
View All 83 Files
Subscribers
imp

Details

Reviewers
melifaro
stevek
Commits
rS331844: Fix memory leaks in route(8).
Summary

route.c flushroutes_fib and interfaces have the same fix.
The local pointer 'buf' is freed before the function returns

Test Plan

Run clang's static analyzer scan-build before to find the problem, run it after to show the problem is resolved.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

trix_juniper.net updated this revision to Diff 25386.Feb 19 2017, 5:52 PM
trix_juniper.net retitled this revision from to Fix memory leaks in route.
trix_juniper.net updated this object.
trix_juniper.net edited the test plan for this revision. (Show Details)
trix_juniper.net added reviewers: stevek, melifaro.
trix_juniper.net set the repository for this revision to rS FreeBSD src repository - subversion.
Herald added a subscriber: imp. · View Herald TranscriptFeb 19 2017, 5:52 PM
melifaro accepted this revision.Mar 31 2018, 3:03 PM
This revision is now accepted and ready to land.Mar 31 2018, 3:03 PM
Closed by commit rS331844: Fix memory leaks in route(8). (authored by melifaro). · Explain WhyMar 31 2018, 3:06 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
sbin/
route/
2 lines

Diff 40945

View Options

head/sbin/route/route.c

Loading...