[cifs-protocol] 117052515795450 Documentation of pre-computation hash in WDigest property is wrong

[Andrew Bartlett]

On Tue, 2017-05-30 at 18:18 +0000, Sreekanth Nadendla wrote:
> Hello Andrew,  MS-SAMR section 3.1.1.8.11.3.1 WDIGEST_CREDENTIALS
> Construction (https://msdn.microsoft.com/en-us/library/cc245680.aspx)
> shows how the 29 different hash values are calculated.
>  
> Section 2.2.10.3 Primary:WDigest - WDIGEST_CREDENTIALS shows the
> order in which those hash values ought to be stored in
> WDIGEST_CREDENTIALS structure. It shows hashes 1 through 29.
>  
> Are you saying that from your findings, you see hash 29 stored first
> and hash 1 stored last in the WDIGEST_CREDENTIALS structure?

No, it is more muddled than that.  Gary (CC'ed) has the details.

>  
> ISSUE:
>  
> WDigest package of supplementalCredentials attribute. Documentation
> of pre-computation hash in WDigest property is wrong. Construction is
> inverted. Needs to fix the document [MS-SAMR]
>  
> 3.1.1.8.11.3 Primary:WDigest Property
> https://msdn.microsoft.com/en-us/library/cc245679.aspx
>  
> 3.1.1.8.11.3.1 WDIGEST_CREDENTIALS Construction
> https://msdn.microsoft.com/en-us/library/cc245680.aspx
>  
>  
>  
> Regards,
> Sreekanth Nadendla
> Microsoft Windows Open Specifications
>  
>  
>  
-- 
Andrew Bartlett
https://samba.org/~abartlet/
Authentication Developer, Samba Team         https://samba.org
Samba Development and Support, Catalyst IT   
https://catalyst.net.nz/services/samba