That seems to work well. Thanks Hans.
On 5/3/19 10:39 pm, Hans Leidekker wrote:
On Tue, 2019-03-05 at 11:42 +0100, Hans Leidekker wrote:
On Tue, 2019-03-05 at 21:23 +1100, Brendan McGrath wrote:
My patch to the mailing list is based on a change I made to my local build to fix an issue while running .NET Core. I can see in the .NET Core code, that it passes BCRYPT_HASH_REUSABLE_FLAG to BCryptCreateHash and expects STATUS_INVALID_PARAMETER if the OS is pre-Win8 (I'm using Win7). That code can be seen here (it's under the MIT licence): https://github.com/dotnet/corefx/blob/master/src/Common/src/Internal/Cryptog...
So until BCRYPT_HASH_REUSABLE_FLAG is implemented - maybe it is better to return STATUS_INVALID_PARAMETER?
Sure, if you restrict it to BCRYPT_HASH_REUSABLE_FLAG. But implementing the flag would be even better, and doesn't look too hard.
Here's a draft patch, can you give it a try?