+ Lohith On 10/04/16 at 01:53P, Gleb Smirnoff wrote: > 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.bel...@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? >
Cheers, Hiren
pgp8eVxXFJl1H.pgp
Description: PGP signature