Page MenuHomeFreeBSD
Differential D9850

Fix 'last' memory leak
ClosedPublic
Actions

Authored by trix_juniper.net on Mar 2 2017, 12:06 AM.
Tags
None
Referenced Files
F154564291: D9850.id29161.diff
Tue, Apr 28, 10:25 PM
F154561554: D9850.id29161.diff
Tue, Apr 28, 10:19 PM
F154550789: D9850.id25854.diff
Tue, Apr 28, 9:33 PM
F154548948: D9850.id25854.diff
Tue, Apr 28, 9:28 PM
Unknown Object (File)
Sat, Apr 25, 10:39 AM
Unknown Object (File)
Thu, Apr 23, 1:50 AM
Unknown Object (File)
Mon, Apr 20, 10:01 AM
Unknown Object (File)
Sat, Apr 18, 1:06 PM
View All Files
Subscribers
imp

Details

Reviewers
stevek
ed
Commits
rS319508: Fix a memory leak with last
Summary

Fix a memory leak with last

last.c wtmp : free memory allocated to 'buf

Test Plan

Run clang's static analyzer 'scan-build' to find the problem and to show it's resolution.

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 25854.Mar 2 2017, 12:06 AM
trix_juniper.net retitled this revision from to Fix 'last' memory leak .
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 TranscriptMar 2 2017, 12:06 AM
ed accepted this revision.Mar 2 2017, 9:17 AM
ed edited edge metadata.
This revision is now accepted and ready to land.Mar 2 2017, 9:17 AM
Closed by commit rS319508: Fix a memory leak with last (authored by stevek). · Explain WhyJun 2 2017, 8:25 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
usr.bin/
last/
2 lines

Diff 29161

View Options

head/usr.bin/last/last.c

Loading...