[PATCH] Bug fixes for libgsmd-tool -m shell

Jorgen Cederlof jc at lysator.liu.se
Wed Jul 23 10:47:37 CEST 2008


I'm not subscribed to this list but just noticed the replies in the
archive. Try to include me as Cc or To.

> So for a string like "R=\0" you assume that strlen("\0") is 0? Took
> me a minute to figure that out. :)

Yeah, "R=\0" is the same string as "R=".

>> +                       } else if (!strncmp(buf, "prr=", 3) &&
>> +                                  (ptr = strchr(buf, '=')) &&
>> +                                  (fcomma = strchr(ptr+1, ','))) {

> strlen("prr=") == 4 but you use strncmp with 3?

Good catch, that's a bug.

> Hey,
> wow reviewing C string handling is no pleasure.

:)

> From the five minute review I think there is only this one strncmp
> mistake. I would need to do a more careful review of all the fcomma,
> lcomma assignments but your patch definitely cleans up the handling
> and I will be happy to apply it once we talked about this one
> strncmp.

Good. Yeah, that strncmp was a mistake, it should have been 4.

> Have you considered using getopts of GNU to do the command line
> parsing?

Nah, that's above my current level of ambition. I'm just a user who
bought a phone as soon as they were available, was unable to choose
operator, and fixed the bug. I changed the rest since the problem was
basically the same, but I doubt I'll do much more until another itch
needs scratching.

  Jörgen




More information about the devel mailing list