Re: [evolution-patches] Re: Exchange connector: Patch for #72801

[Sarfraaz Ahmed <asarfraaz novell com>]

On Fri, 2005-03-11 at 11:24 +0530, Sushma Rai wrote:
> We can not do that ... because e2k_autoconfig_get_context() is what
> sets nt_domain, if autoconfiguration is not done, which is most common.
Hmm, fine. Try avoiding the re-assignment of nt_domain to account by checking for its existance.

-- Sarfraaz

> -Sushma.
>
> On Fri, 2005-03-11 at 10:58 +0530, Sarfraaz Ahmed wrote:
> > Looks fine. In the exchange_account_connect, we should rather be
> > setting the nt_domain before the "try_password_again" label, just
> > after we do a e2k_autoconfig_new.
> > 
> > Thanks
> > -- Sarfraaz
> > 
> > > @@ -1326,6 +1347,11 @@ exchange_account_connect (ExchangeAccoun
> > >  
> > >   try_connect_again:
> > >  	account->priv->ctx = e2k_autoconfig_get_context (ac, NULL, &result);
> > > +
> > > +	if (ac->nt_domain)
> > > +		account->priv->nt_domain = g_strdup (ac->nt_domain);
> > > +	else
> > > +		account->priv->nt_domain = NULL;
> > >  
> > >  	if (result != E2K_AUTOCONFIG_OK) {
> >