FreeTDS Development Group

[Brian Bruns <camber AT ais.org>]

Ok,

Looking it over tds7_send_login computes the strlen() three times for each 
of the login fields.  So i moved all the strlen()'s up top and checked for 
null there.  Changes are in CVS if someone cares to try it.

Brian


On Mon, 5 Nov 2001, Franco Gasperino wrote:

> On Monday 05 November 2001 13:44, you wrote:
> > I think we should fix the problem properly.
> >
> > I'm not confident that future maintainers of the code will realise that
> > strlen is different.  At least we should change it to STRLEN.
> >
> > Can we have backtraces of where the strlen()'s were given NULL?
> 
>   As of last nite CVS, the problem still existed with tds7_send_login(), 
> and 
> config->app_name & config->library being NULL when used from iODBC.
> 
>