On Thursday 27 December 2007 05:38, John Drescher wrote:
> At the end of the file (bnet_server.c), I believe the len param should
> be the third param in the calls to bstrncpy and bstrncat and not the
> second.
>
> len = strlen(caller) + strlen(who) + 3;
>       buf = (char *)malloc(len);
>       bstrncpy(buf, len, who);
>       bstrncat(buf, len, ": ");
>       bstrncat(buf, len, caller);
>       bs = init_bsock(NULL, newsockfd, _("client"), buf, bsock->port,
> &cli_addr);
>       free(buf);
>       return bs;
>    }
> }

Yes, you are absolutely correct.  Fortunately, the code is turned off because 
it is never used, which is why we never see any compiler errors.  Rather than 
fixing it, since there are likely to be other errors, I will remove it for 
the next version ...

Best regards,

Kern

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Bacula-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/bacula-devel

Reply via email to