Re: [PATCH 2/2] ext4: add sanity checks in __ext4_check_dir_entry

On Mon, Jun 03, 2013 at 01:00:16PM +0300, Ruslan Bilovol wrote:

Added checks for NULL before dereferencing some pointers

+	if (!de || !bh || !dir)
+		return 1;

Do you know which one of these pointers was NULL?

I want't make sure we're fixing the root issue as opposed to papering
over it.  I'm not seeing how any of these pointers could have been
NULL (at least in my upstream kernel), and while I believe that this
helped for your kernel, I want to make sure we understand exactly what
was going on.

Thanks,

					- Ted