Re: [Freeipa-devel] [PATCH] 480 Do not store LastPwdChange unless it really changed

2012-02-15 Thread Rob Crittenden
Rob Crittenden wrote: Simo Sorce wrote: Due to an idiosyncrasy of kadmin, the right flag to indicate krbLastPwdChange is changed is not set. The previous check ended up always saving the data in all cases because the data was always present. Restrict it to store a password change when there is a

Re: [Freeipa-devel] [PATCH] 480 Do not store LastPwdChange unless it really changed

2012-02-15 Thread Rob Crittenden
Simo Sorce wrote: Due to an idiosyncrasy of kadmin, the right flag to indicate krbLastPwdChange is changed is not set. The previous check ended up always saving the data in all cases because the data was always present. Restrict it to store a password change when there is actually new key materia

[Freeipa-devel] [PATCH] 480 Do not store LastPwdChange unless it really changed

2012-02-14 Thread Simo Sorce
Due to an idiosyncrasy of kadmin, the right flag to indicate krbLastPwdChange is changed is not set. The previous check ended up always saving the data in all cases because the data was always present. Restrict it to store a password change when there is actually new key material. This prevents al