=?iso-8859-1?Q?=E2=9C=98ntpqMime-Version: 1.0

Hal Murray hmurray at megapathdsl.net
Tue May 31 06:38:09 UTC 2016


> I'll track this down tomorrow.  I have a pretty good idea where it might be.

The immediate problem is a signed/unsigned mixup.  argc has gone negative and 
a (u_int) that I probably added to squash compiler warnings isn't bailing on 
the loop that's grabbing the rest of the arguments as host names.

If I fix that, it dies later on trying to check the first char of the host 
name for "[" because the test for no-args so we should supply a default 
hostname used 0 == argc and didn't setup anything.

So backing up, I think the real problem is the way the command line options 
parser is handling missing arguments.  I think it's setting ntp_optind to 2 
when the second part isn't there.

There is also the problem of a bogus error message.  A missing argument turns 
into an unknown command message.

I'll let you sort that out.




-- 
These are my opinions.  I hate spam.





More information about the devel mailing list