From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from out1-smtp.messagingengine.com ([66.111.4.25]:58563 "EHLO out1-smtp.messagingengine.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726655AbeLSAFm (ORCPT ); Tue, 18 Dec 2018 19:05:42 -0500 Message-ID: <941eac5115a41a0bfe8736816b7600ddfcc0371e.camel@themaw.net> Subject: Re: kernel BUG at fs/inode.c:LINE! From: Ian Kent To: Andrew Morton Cc: Al Viro , syzbot , DmitryVyukov , linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org, syzkaller-bugs@googlegroups.com Date: Wed, 19 Dec 2018 08:05:35 +0800 In-Reply-To: <20181218130958.88ac9d638c717a1ab1db9dc3@linux-foundation.org> References: <00000000000051e9c2057d31a563@google.com> <20181217072144.GQ2217@ZenIV.linux.org.uk> <95ae4c9893c89189d4309fe673ade6f389280101.camel@themaw.net> <66d497c00cffb3e4109ca0d5287c8277954d7132.camel@themaw.net> <20181218130958.88ac9d638c717a1ab1db9dc3@linux-foundation.org> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On Tue, 2018-12-18 at 13:09 -0800, Andrew Morton wrote: > On Tue, 18 Dec 2018 19:34:57 +0800 Ian Kent wrote: > > > > See > > > https://www.ozlabs.org/~akpm/mmotm/broken-out/autofs-fix-possible-inode-leak-in-autofs_fill_super.patch > > > > > > I think this will fix it, I'll forward it to Andrew if you agree: > > > > Actually, looking at it again the above patch is plain not needed, > > dropping it and updating the patch which follows it in the series > > is what needs to be done. > > > > Andrew, what should I do to make this easiest for you to handle, > > a respost with v2 in the subject of the patch affected by dropping > > the above patch? > > I dropped the patch and fixed up the fallout. Thanks Andrew, much appreciated. > > The patch wasn't true anyway. "There is no check at all for a failure > to allocate the root inode in autofs_fill_super(), handle it." In fact, > d_make_root(NULL) will just return NULL and autofs_fill_super() handles > that appropriately. The not so funny thing is that I'm sure I looked at this some time in the distant past and saw how d_make_root() behaved. The lesson for me is don't try and fix other things seen while working on something else, return later and do it properly. > > However let's note that when autofs_get_inode() or d_make_root() fail, > autofs_fill_super() will return -EINVAL. Should have been -ENOMEM, I > guess? > That's right, but I don't think that's urgent so I'll send a patch for it after the coming merge window. The strictexpire option addition is urgent for me so I don't want to upset any chance of that being merged sooner rather than later. Ian