Fix a memory leak in ppp
ClosedPublic
Actions

Authored by trix_juniper.net on Mar 2 2017, 1:41 AM.

Tags

None

Referenced Files

	F168154746: D9852.id29286.diff
	Wed, Aug 26, 3:22 PM

	F168153726: D9852.diff
	Wed, Aug 26, 3:15 PM

	Unknown Object (File)
	Sat, Aug 22, 3:57 AM

	Unknown Object (File)
	Fri, Aug 21, 12:53 AM

	Unknown Object (File)
	Fri, Aug 21, 12:53 AM

	Unknown Object (File)
	Wed, Aug 19, 12:22 PM

	Unknown Object (File)
	Tue, Aug 11, 7:11 AM

	Unknown Object (File)
	Tue, Aug 4, 6:02 AM

View All Files

Subscribers

imp

Details

Reviewers

stevek

Commits

rS319638: Before returning because of an memory allocation error, free the memory

Summary

ppp iface.c
Before returning because of an memory allocation error, free the memory already allocated to buf.

Test Plan

Run clang's static analyzer, scan-build, to find the problem and to show it 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 25855.Mar 2 2017, 1:41 AM

trix_juniper.net retitled this revision from to Fix a memory leak in ppp.

trix_juniper.net updated this object.

trix_juniper.net edited the test plan for this revision. (Show Details)

trix_juniper.net added a reviewer: stevek.

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, 1:41 AM

Looks reasonable to me.

This revision is now accepted and ready to land.Mar 2 2017, 10:39 PM

Closed by commit rS319638: Before returning because of an memory allocation error, free the memory (authored by stevek). · Explain WhyJun 6 2017, 9:51 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

usr.sbin/

ppp/

iface.c

1 line

Diff 29286

View Options

Fix a memory leak in pppClosedPublicActions