On Fri, Apr 08, 2016 at 11:43:49AM +0100, David CARLIER wrote:
> Hi,
> 
> Here this is a tiny change, hope it finds its way.
> 
> Kind regards.

> From 321074b31d9561ea1af51c65c9a912f8a08f555c Mon Sep 17 00:00:00 2001
> From: David Carlier <dcarl...@afilias.info>
> Date: Fri, 8 Apr 2016 10:37:02 +0100
> Subject: [PATCH 3/4] CLEANUP: sample: initialize the pointer before
>  parse_binary call.
> 
> parse_binary line 2025 checks the nullity of binstr parameter.
> Other calls of parse_binary properly zeroify this parameter.

Actually I'll change the commit subject to BUG/MEDIUM since it randomly
prevents the const parser from working at all. Most of the time the
if (!*binstr) will fail and the check will randomly end up failing on
if (!*binstrlen) which is not initialized.

Thanks!
Willy


Reply via email to