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
F102751840: D9850.id29161.diff
Sat, Nov 16, 4:52 PM
Unknown Object (File)
Thu, Nov 14, 8:38 AM
Unknown Object (File)
Sun, Nov 10, 8:33 PM
Unknown Object (File)
Tue, Oct 22, 7:26 AM
Unknown Object (File)
Tue, Oct 22, 3:50 AM
Unknown Object (File)
Oct 5 2024, 3:55 PM
Unknown Object (File)
Oct 2 2024, 4:02 PM
Unknown Object (File)
Sep 23 2024, 2:00 PM
View All 53 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...