Re: [PATCH] fuse: avoid clobbering fc->release with fuse_conn_init(fc)

From: Jeremiah Mahler
Date: Mon Jun 01 2015 - 13:28:57 EST


Miklos,

On Mon, Jun 01, 2015 at 02:03:02PM +0200, Miklos Szeredi wrote:
> On Mon, Jun 1, 2015 at 3:31 AM, Jeremiah Mahler <jmmahler@xxxxxxxxx> wrote:
> > fuse_conn_init(fc) zeros the entire fc structure before it assigns
> > specific values. Since fc->release is set before fuse_conn_init(fc)
> > is called, the value gets set to zero, which results in a NULL pointer
> > deference when fc->release() is called. Fix the problem by assigning
> > fc->release after calling fuse_conn_init(fc) instead of before.
>
> I added this fix to the original patch and force pushed to
> fuse.git/for-next. Better keep it in one so that your fix is not left
> out accidentally when being backported to -stable.
>
> Thanks,
> Miklos
>

Good idea, thanks :-)

--
- Jeremiah Mahler
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/