Re: 5.51 - NSE assertion - timeout_msecs

[Patrick Donnelly <batrick () batbytes com>]

On Tue, Apr 5, 2011 at 3:31 PM, David Fifield <david () bamsoftware com> wrote:
> On Tue, Apr 05, 2011 at 03:30:10PM -0400, Patrick Donnelly wrote:
> > Well the code says:
> >
> >   nu->timeout = luaL_checkint(L, 2);
> >   if ((int) nu->timeout < -1)
> >     return luaL_error(L, "Negative timeout: %d", nu->timeout);
> >
> > Should that be "<= -1" or "< 0"?
>
> No, -1 means no timeout. It's a special case in msevent_new.

I thought so. I'll add a comment to that code to prevent future confusion.

-- 
- Patrick Donnelly
_______________________________________________
Sent through the nmap-dev mailing list
http://cgi.insecure.org/mailman/listinfo/nmap-dev
Archived at http://seclists.org/nmap-dev/