Re: [PATCH] mtdinfo: don't open NULL pointer when getting region_info with `-a'

Brian Norris
2011-08-05T22:59:26Z

Hi (other) Brian,

On Fri, Aug 5, 2011 at 12:29 AM, Brian Foster <brian.foster@maxim-ic.com> wrote:
>  I suggest adding some round brackets:
>
>        if (!args.node || (!args.map && mtd->region_cnt == 0))
>
>  Less confusing.  Otherwise, looks Ok
>  to me (not actually built or tested).

Thanks for the comment. But I'm guessing this patch isn't the best
idea after all (unless Mike's plan doesn't work).

Anyway, it'd probably help if you test the final product, since you're
the one who actually sees the error!

Thanks,
Brian