Fix memory leak in sddaregister().
ClosedPublic
Actions

Authored by bz on Aug 7 2020, 11:50 AM.

Tags

None

Referenced Files

	F172918036: D25987.id75590.diff
	Tue, Sep 22, 2:19 AM

	F172917689: D25987.id75552.diff
	Tue, Sep 22, 2:15 AM

	F172917526: D25987.diff
	Tue, Sep 22, 2:14 AM

	Unknown Object (File)
	Fri, Sep 18, 4:21 AM

	Unknown Object (File)
	Wed, Sep 9, 10:51 PM

	Unknown Object (File)
	Wed, Sep 9, 10:19 PM

	Unknown Object (File)
	Sun, Sep 6, 3:36 PM

	Unknown Object (File)
	Sun, Sep 6, 2:07 PM

View All Files

Subscribers

None

Details

Reviewers

kibab
manu
imp
scottl

Group Reviewers

cam

Commits

rS364043: mmc_da: fix memory leak in sddaregister()

Summary

In case we are failing to allocate mmcdata, we are returning with
a softc allocated but not attached to anything and thus leak the
memory.

MFC after: 2 weeks
X-MFC: only if we also mfc other mmccam changes?

Diff Detail

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 32834
Build 30245: arc lint + arc unit

Event Timeline

bz created this revision.Aug 7 2020, 11:50 AM

Herald added a reviewer: cam. · View Herald TranscriptAug 7 2020, 11:50 AM

bz requested review of this revision.Aug 7 2020, 11:50 AM

Harbormaster completed remote builds in B32834: Diff 75552.Aug 7 2020, 11:50 AM

scottl accepted this revision.Aug 7 2020, 6:38 PM

This revision is now accepted and ready to land.Aug 7 2020, 6:38 PM

Closed by commit rS364043: mmc_da: fix memory leak in sddaregister() (authored by bz). · Explain WhyAug 7 2020, 7:58 PM

This revision was automatically updated to reflect the committed changes.

bz added a commit: rS364043: mmc_da: fix memory leak in sddaregister().

Revision Contents
Changeset List

Path

Size

sys/

cam/

mmc/

mmc_da.c

2 lines

Diff 75552

View Options

Fix memory leak in sddaregister().ClosedPublicActions