svn commit: r306337 - head/sys/kern

Gleb Smirnoff glebius at FreeBSD.org
Tue Oct 4 20:53:54 UTC 2016


  Hiren,

On Mon, Sep 26, 2016 at 10:13:58AM +0000, Hiren Panchasara wrote:
H> Author: hiren
H> Date: Mon Sep 26 10:13:58 2016
H> New Revision: 306337
H> URL: https://svnweb.freebsd.org/changeset/base/306337
H> 
H> Log:
H>   In sendit(), if mp->msg_control is present, then in sockargs() we are allocating
H>   mbuf to store mp->msg_control. Later in kern_sendit(), call to getsock_cap(),
H>   will check validity of file pointer passed, if this fails EBADF is returned but
H>   mbuf allocated in sockargs() is not freed. Fix this possible leak.
H>   
H>   Submitted by:	Lohith Bellad <lohith.bellad at me.com>
H>   Reviewed by:	adrian
H>   MFC after:	3 weeks
H>   Differential Revision:	https://reviews.freebsd.org/D7910

The commit appeared to be incorrect, but a problem exists. I'd like to look at it.
Is there any reproduce recipe for the leak or bug filed?

-- 
Totus tuus, Glebius.


More information about the svn-src-head mailing list