Hi Rabin,

On 29 October 2014 16:21, Rabin Vincent <[email protected]> wrote:
> Hush segfaults if it sees a syntax error while attempting to parse a
> command:
>
>  $ ./u-boot -c "'"
>  ...
>  syntax error
>  Segmentation fault (core dumped)
>
> This is due to a NULL pointer dereference of in_str->p in static_peek().
> The problem is that the exit condition for the loop in
> parse_stream_outer() checks for rcode not being -1, but rcode is only
> ever 0 or 1.
>
> Signed-off-by: Rabin Vincent <[email protected]>

Acked-by: Simon Glass <[email protected])
Tested-by: Simon Glass <[email protected])

BTW I notice that I still get a crash with

./u-boot -c "''"

Regards,
Simon
_______________________________________________
U-Boot mailing list
[email protected]
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to