[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: smbk5pwd crash for missing symbol




28 maj 2008 kl. 09.08 skrev Howard Chu:

> Love Hörnquist Åstrand wrote:
>> 28 maj 2008 kl. 02.57 skrev Guillaume Rousse:
>>
>>> +		hdb_entry_set_pw_change_time(context,&ent, 0);
>>> +
>>> +		if (krb5_config_get_bool_default(context, NULL, FALSE,
>>> +			"kadmin", "save-password", NULL)) {
>>> +			ret = hdb_entry_set_password(context,
>>> +				context->db,&ent, qpw->rs_new.bv_val);
>>> +			if (ret != 0)
>>> +				break;
>>> +		}
>>
>> As Howard said, you can drop this part. It needs to be done  
>> diffrenlty
>> with the hdb backend.
>>
>> Other then that the patch looks fine.
>
> Given that this behavior is now version-dependent, it would be nice  
> if the Heimdal header files provided a preprocessor macro with the  
> version number, instead of just the heimdal_version string variables.

The behavior have always been version dependent for private variable  
openldap used (prefixed with _ and not included in public header files).

I need to covert the string version into a cpp number though in  
configure (see the end of configure.in).

Love