bug in HTHost_decreaseRetry ???

I just started working with this code base and am unsure about where the
patches are.  I noticed one collection on the website.  I'm assuming
that 5.1b has all the patches applied from 5.1a.

Version 5.1b ....
The code in HTHost_decreaseRetry looks wrong, and causes an infinite
loop in HTDoConnect.  Notice that the host->retry count doesn't get
decreased unless host is NULL which would normally cause an access
violation anyway :( 

Suggest  the ! should be removed from !host.

PUBLIC BOOL HTHost_decreaseRetry (HTHost * host)
{
    if (!host) {
	if (host->retry > 0) host->retry--;
	return YES;
    }
    return NO;
}

PUBLIC int HTHost_retry (HTHost * host){
    if (!host) return 0;
    return host->retry;
}

-- 
------------------------------------------------------------------------
John Nunneley                                Phone: +1 408 457 4510
DASCOM Inc.                                  Fax:   +1 408 457 0710
1509 Seabright Avenue                        Email: johnn@dascom.com
Santa Cruz CA 95062                          WWW: http://www.dascom.com/
------------------------------------------------------------------------

Received on Friday, 13 June 1997 15:06:00 UTC