Page MenuHomeFreeBSD

Fix memory issues with nvi
Needs ReviewPublic

Authored by trix_juniper.net on Feb 23 2017, 11:28 PM.
Tags
None
Referenced Files
Unknown Object (File)
Dec 20 2023, 1:44 AM
Unknown Object (File)
Oct 25 2023, 9:58 AM
Unknown Object (File)
Sep 16 2023, 1:25 AM
Unknown Object (File)
Sep 6 2023, 3:37 AM
Unknown Object (File)
Jul 23 2023, 11:33 PM
Unknown Object (File)
Jun 7 2023, 10:33 AM
Unknown Object (File)
Apr 18 2023, 9:48 PM
Unknown Object (File)
Jan 13 2023, 4:08 PM
Subscribers

Details

Reviewers
bapt
stevek
Summary

ex_cscope.c cscope_find fix a double free of the variable np by poisoning it after it is freed.

exf.c file_backup fix a memory leak by freeing the variable d.

Test Plan

Run clang's static analyzer, scan-build, before to reproduce the problems. Run scan-build after to verify resolution.

Diff Detail

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

Event Timeline

trix_juniper.net retitled this revision from to Fix memory issues with nvi.
trix_juniper.net updated this object.
trix_juniper.net edited the test plan for this revision. (Show Details)
trix_juniper.net added reviewers: stevek, bapt.
trix_juniper.net set the repository for this revision to rS FreeBSD src repository - subversion.
contrib/nvi/common/exf.c
1191

The test is not needed, free already do that test

Have you tried to upstream that fix? upstream is here: https://github.com/lichray/nvi2