[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20110916225332.GJ16246@thunk.org>
Date: Fri, 16 Sep 2011 18:53:32 -0400
From: Ted Ts'o <tytso@....edu>
To: Eric Sandeen <sandeen@...hat.com>
Cc: linux-ext4@...r.kernel.org
Subject: Re: [PATCH 07/25] e2fsck: handle null fs in print_pathname()
On Fri, Sep 16, 2011 at 03:49:22PM -0500, Eric Sandeen wrote:
> testing fs for NULL in expand_percent_expression():
>
> e2fsck_ctx = fs ? (e2fsck_t) fs->priv_data : NULL;
>
> implies that fs could be NULL, but it's passed to print_pathname()
> which defererences it without further testing.
>
> So make this safe by returning "???" for a nul fs.
>
> Signed-off-by: Eric Sandeen <sandeen@...hat.com>
Applied, thanks.
- Ted
--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists