linux-kernel - Re: [PATCH] fs-writeback: fix NULL pointer dereference in, __mark_inode

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <99773600-FC41-41A9-8544-BF7E09065546@cam.ac.uk>
Date:	Tue, 15 Mar 2011 12:11:42 +0000
From:	Anton Altaparmakov <aia21@....ac.uk>
To:	Torsten Hilbrich <torsten.hilbrich@...unet.com>
Cc:	LKML <linux-kernel@...r.kernel.org>,
	Andreas Bießmann <biessmann@...science.de>,
	Sergey Senozhatsky <sergey.senozhatsky@...il.com>,
	Christoph Hellwig <hch@....de>, Jens Axboe <axboe@...nel.dk>,
	Linus Torvalds <torvalds@...ux-foundation.org>,
	linux-fsdevel <linux-fsdevel@...r.kernel.org>
Subject: Re: [PATCH] fs-writeback: fix NULL pointer dereference in, __mark_inode_dirty

Hi,

On 15 Mar 2011, at 10:17, Torsten Hilbrich wrote:
> On 10.03.2011 11:39, Torsten Hilbrich wrote:
>> I ran into the same problem and successfully applied your fix.
>> 
>> I was able to reproduce this panic and bisected it to the following commit:
>> 
>> commit aaead25b954879e1a708ff2f3602f494c18d20b5
>> Author: Christoph Hellwig <hch@....de>
>> Date:   Mon Oct 4 14:25:33 2010 +0200
>> 
>>    writeback: always use sb->s_bdi for writeback purposes
>> 
>> The steps to reproduce it on my test system (T60p with Intel Core Duo) were.
> 
> Added Christoph to CC. I also open a bug report at https://bugzilla.kernel.org/show_bug.cgi?id=31112

This is already being handled.  It is the same as other bug reports, i.e. the fact that sb->s_bdi is made NULL on device removal and if it happens at the wrong time you then get a NULL pointer dereference.

Jens Axboe just only yesterday posted an initial patch for this.  Can you please test it and report back if it does indeed cure the problem?

The patch can be found here for example:

	https://lkml.org/lkml/2011/3/14/25

Best regards,

	Anton
-- 
Anton Altaparmakov <aia21 at cam.ac.uk> (replace at with @)
Unix Support, Computing Service, University of Cambridge, CB2 3QH, UK
Linux NTFS maintainer, http://www.linux-ntfs.org/

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