Jacek Caban : mshtml: Alloc proper BSTR size in return_nscstr.

Alexandre Julliard julliard at winehq.org
Wed Mar 27 17:27:37 CDT 2019


Module: wine
Branch: master
Commit: 9f4d05b11c32a5417ef31da8aa1041166d8cfe27
URL:    https://source.winehq.org/git/wine.git/?a=commit;h=9f4d05b11c32a5417ef31da8aa1041166d8cfe27

Author: Jacek Caban <jacek at codeweavers.com>
Date:   Wed Mar 27 18:53:01 2019 +0100

mshtml: Alloc proper BSTR size in return_nscstr.

Signed-off-by: Jacek Caban <jacek at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 dlls/mshtml/tests/xhr.js     | 1 +
 dlls/mshtml/xmlhttprequest.c | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/dlls/mshtml/tests/xhr.js b/dlls/mshtml/tests/xhr.js
index 8c42816..cb7e461 100644
--- a/dlls/mshtml/tests/xhr.js
+++ b/dlls/mshtml/tests/xhr.js
@@ -29,6 +29,7 @@ function test_xhr() {
             next_test();
     }
     var onload_func = xhr.onload = function() {
+        ok(xhr.statusText === "OK", "statusText = " + xhr.statusText);
         if(complete_cnt++)
             next_test();
     };
diff --git a/dlls/mshtml/xmlhttprequest.c b/dlls/mshtml/xmlhttprequest.c
index e235bf9..5f844d5 100644
--- a/dlls/mshtml/xmlhttprequest.c
+++ b/dlls/mshtml/xmlhttprequest.c
@@ -80,7 +80,7 @@ static HRESULT return_nscstr(nsresult nsres, nsACString *nscstr, BSTR *p)
 
     if(*str) {
         len = MultiByteToWideChar(CP_UTF8, 0, str, -1, NULL, 0);
-        *p = SysAllocStringLen(NULL, len);
+        *p = SysAllocStringLen(NULL, len - 1);
         if(!*p) {
             nsACString_Finish(nscstr);
             return E_OUTOFMEMORY;




More information about the wine-cvs mailing list