[lwip-devel] [patch #6765] Supporting new line characters in inet_aton()

[Simon Goldschmidt]

Follow-up Comment #6, patch #6765 (project lwip):

Sorry, seems like I messed that up. I converted

if (c != ' ' && (!isprint(c) || !isspace(c)))

to

if (c != ' ' && !isprint(c) && !isspace(c))

since the first version returned always 'true' if c != 0.

if (c != ' ' && !isspace(c))

seems to give the correct results, tested for:

"abc.de"
"127.0.0.1"
"127.0.0.1 "
"127.0.0.1n"
"127.0.0.1r"
"3com.com"
"3com.com "
"3com.com n"
"3com.comn"
"3com.comr"
"3com.comrn"
"3com.comrn"
"3com.comnr"

Anyone something against that?

    _______________________________________________________

Reply to this item at:

  <http://savannah.nongnu.org/patch/?6765>

_______________________________________________
  Nachricht geschickt von/durch Savannah
  http://savannah.nongnu.org/