Hi Isaac !

Your program will fail on lines starting with the word server
(eg. serverxyz), that is it does not check for clear word
boundary and gives wrong results in that case.

>while (cbuf[i] > 35) i++;

Unwise to do this in a not poor ASCII environment, as most
systems are nowadays. This way you allow unprintable and any
kind of illegal characters in time server addresses. ... and what
about buffer overflow? Won't this loop then run to unknown
locations?

Beside this: Make it a default NO configuration, not being
included in binaries build from standard options.

--
Harald
_______________________________________________
busybox mailing list
busybox@busybox.net
http://lists.busybox.net/mailman/listinfo/busybox

Reply via email to