[U-Boot] [PATCH 1/5] Check for NULL prompt in readline_into_buffer()

Peter Tyser ptyser at xes-inc.com
Sat Oct 24 23:37:47 CEST 2009


Wolfgang Denk wrote:
> Dear Peter Tyser,
> 
> In message <1256259563-32725-1-git-send-email-ptyser at xes-inc.com> you wrote:
>> Previously, passing readline() or readline_into_buffer() a NULL 'prompt'
>> parameter would result in puts() printing garbage when
>> CONFIG_CMDLINE_EDITING was enabled.
> 
> What would be the situation when this happens? I cannot see easily how
> to trigger this bug?

I don't think anything in current code would trigger this.  If some 
boards enabled CONFIG_CMDLINE_EDITING, they would have issues, eg 
board/eltec/bab7xx/misc.c.

So its not fixing a bug, just preventing a future one:)  It was also 
necessary to implement the 5/5 patch "Add 'editenv' command" which is 
how I triggered the "future bug".

In this series, patches 1/2/4 are non-critical cleanup/fixes.  I didn't 
plan on them get merged for the upcoming release, but I think they 
should get picked up at some point.  Patches 3/5 add new features - 
basically support for a new 'editenv' command.  These changes are more 
debatable, so we can discuss if you'd like to include these or not.

Best,
Peter


More information about the U-Boot mailing list