Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 04 Apr 2016 10:24:31 +0000
From:      bugzilla-noreply@freebsd.org
To:        freebsd-bugs@FreeBSD.org
Subject:   [Bug 207626] [cam] Memory leak in ctl.c
Message-ID:  <bug-207626-8-dF8ocWhZpR@https.bugs.freebsd.org/bugzilla/>
In-Reply-To: <bug-207626-8@https.bugs.freebsd.org/bugzilla/>
References:  <bug-207626-8@https.bugs.freebsd.org/bugzilla/>

next in thread | previous in thread | raw e-mail | index | archive | help
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D207626

--- Comment #4 from CTurt <cturt@hardenedbsd.org> ---
Shawn, in your patch, setting `tmpptr` to `NULL` after it has been freed is
unnecessary since it is out of scope of the `bailout` label, so nothing can=
 use
the freed pointer to cause use after free, after the `goto`; however it may=
 be
preferable to do so out of good practice.

--=20
You are receiving this mail because:
You are the assignee for the bug.=



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?bug-207626-8-dF8ocWhZpR>