Page MenuHomeFreeBSD
Differential D9652

fix memory leak in 'finger'
ClosedPublic
Actions

Authored by trix_juniper.net on Feb 17 2017, 5:50 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Nov 25, 6:04 PM
Unknown Object (File)
Tue, Nov 25, 4:13 AM
Unknown Object (File)
Mon, Nov 24, 9:04 PM
Unknown Object (File)
Mon, Nov 24, 12:39 PM
Unknown Object (File)
Sat, Nov 22, 5:26 PM
Unknown Object (File)
Tue, Nov 18, 7:40 PM
Unknown Object (File)
Sun, Nov 9, 9:26 AM
Unknown Object (File)
Thu, Nov 6, 4:14 AM
View All 95 Files
Subscribers
imp

Details

Reviewers
stevek
ed
Commits
rS319903: The variable nargv is allocated but never freed, so free it when the it
Summary

the variable nargv is allocated but never freed, so free it when the it is no longer used

Test Plan

Run clang's scan-build before the fix to find the problem, run after to verify fixed.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

trix_juniper.net updated this revision to Diff 25319.Feb 17 2017, 5:50 PM
trix_juniper.net retitled this revision from to fix memory leak in 'finger'.
trix_juniper.net updated this object.
trix_juniper.net edited the test plan for this revision. (Show Details)
trix_juniper.net added reviewers: stevek, ed.
trix_juniper.net set the repository for this revision to rS FreeBSD src repository - subversion.
Herald added a subscriber: imp. · View Herald TranscriptFeb 17 2017, 5:50 PM
ed accepted this revision.Feb 17 2017, 5:53 PM
ed edited edge metadata.
This revision is now accepted and ready to land.Feb 17 2017, 5:53 PM
Closed by commit rS319903: The variable nargv is allocated but never freed, so free it when the it (authored by stevek). · Explain WhyJun 13 2017, 3:50 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
usr.bin/
finger/
1 line

Diff 25319

View Options

usr.bin/finger/finger.c

Loading...