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

Re: smbk5pwd crash for missing symbol

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.
-- 
   -- Howard Chu
   CTO, Symas Corp.           http://www.symas.com
   Director, Highland Sun     http://highlandsun.com/hyc/
   Chief Architect, OpenLDAP  http://www.openldap.org/project/