]> granicus.if.org Git - curl/commit
schannel: client certificate store opening fix
authorIhor Karpenko <ihor.karpenko@gmail.com>
Thu, 23 Aug 2018 11:18:17 +0000 (14:18 +0300)
committerDaniel Stenberg <daniel@haxx.se>
Fri, 24 Aug 2018 07:03:28 +0000 (09:03 +0200)
commit6b6c2b8d57a69a256f7a727784876d8cc37aa669
tree39db8395d778abc99f63aea437432d31d7e709c6
parent8f3c3cd08a5b252002a4abfb19780850fc51040e
schannel: client certificate store opening fix

1) Using CERT_STORE_OPEN_EXISTING_FLAG ( or CERT_STORE_READONLY_FLAG )
while opening certificate store would be sufficient in this scenario and
less-demanding in sense of required user credentials ( for example,
IIS_IUSRS will get "Access Denied" 0x05 error for existing CertOpenStore
call without any of flags mentioned above ),

2) as 'cert_store_name' is a DWORD, attempt to format its value like a
string ( in "Failed to open cert store" error message ) will throw null
pointer exception

3) adding GetLastError(), in my opinion, will make error message more
useful.

Bug: https://curl.haxx.se/mail/lib-2018-08/0198.html

Closes #2909
lib/vtls/schannel.c