patch: fix trivial gdbserver arg parsing buglet

Daniel Jacobowitz drow@false.org
Tue Mar 27 19:10:00 GMT 2007


On Wed, Mar 14, 2007 at 09:02:00PM -0400, Mat Hostetter wrote:
> The standard strtoul idiom is that strtoul succeeds when *arg_end == '\0'
> and the input string is not empty.
> 
> This gdbserver code is incorrectly checking to see if the input
> string is empty by comparing the string pointer itself to '\0'
> rather than its first character.
> 
> This bug is pretty harmless, but what the heck.

Whoops.  Thanks for the patch; I checked it in.

-- 
Daniel Jacobowitz
CodeSourcery



More information about the Gdb-patches mailing list