Re: [PATCH 3/5] pstore: handle zero-sized prz in series

From: Mark Salyzyn
Date: Tue Jan 13 2015 - 18:42:48 EST


On 01/13/2015 03:08 PM, Kees Cook wrote:
On Tue, Jan 13, 2015 at 2:32 PM, Mark Salyzyn <salyzyn@xxxxxxxxxxx> wrote:
ramoops_pstore_read fails to return the next in a prz
series after first zero-sized entry, not venturing to
the next non-zero entry.

Signed-off-by: Mark Salyzyn <salyzyn@xxxxxxxxxxx>
This seems fine. Out of curiosity, when was the bad behavior this encountered?
During integration testing of the pmsg interface, we found that if any (most notably ftrace) of the previous prz's were zero in length. Before the addition of pmsg it would have occurred if you zero'd the console size, but had ftrace set; an unlikely scenario _until_ pmsg was added.

Acked-by: Kees Cook <keescook@xxxxxxxxxxxx>

Thanks!

-Kees

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