[Bug 26340] New: MSXML3, HTTP POST fails
http://bugs.winehq.org/show_bug.cgi?id=26340 Summary: MSXML3, HTTP POST fails Product: Wine Version: 1.3.15 Platform: x86 URL: http://netikka.net/dev/upload.exe OS/Version: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: msxml3 AssignedTo: wine-bugs(a)winehq.org ReportedBy: ocean04(a)suomi24.fi Example: Loading photograph (jpg) to photoserver. http://netikka.net/dev/upload.exe Just select some JPG-file and after waiting you should get image html tag in return. But failure Ubuntu 10.10 -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=26340 --- Comment #1 from Nikolay Sivov <bunglehead(a)gmail.com> 2011-03-07 04:17:40 CST --- Attach source code for that application please. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=26340 --- Comment #2 from Nikolay Sivov <bunglehead(a)gmail.com> 2011-03-07 04:31:18 CST --- This one could be msxml bug, from log: --- 0027:trace:msxml:httprequest_send (0x15e090)->({vt 16396}) 0027:trace:msxml:BindStatusCallback_create created callback 0x178888 0027:fixme:msxml:BindStatusCallback_create unsupported body data type 16396 --- so type is VT_BYREF|VT_VARIANT that we don't support for input. Native msxml6 hits a fixme in urlmon instead, probably unrelated thing. Anyway I'm waiting for source to write a test for that. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=26340 --- Comment #3 from ocean04(a)suomi24.fi 2011-03-07 05:14:07 CST --- Created an attachment (id=33556) --> (http://bugs.winehq.org/attachment.cgi?id=33556) POST function I don't have complete source for this example with me right now, but I attached the POST function I use. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=26340 --- Comment #4 from Nikolay Sivov <bunglehead(a)gmail.com> 2011-03-07 05:34:59 CST --- That's enough, thanks. It looks like you use VT_UI1|VT_ARRAY to store POST body, and after that this variant is passed as VT_VARIANT|VT_BYREF to send() method. I'll do some tests for that case, should be simple. By the way, unrelated to this bug but still. It's better to avoid wait loops like you use, it's designed to be async so you can set your callback with onreadystatechange() and use another timer to track timeout. It will be a nice test for our implementation as well. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=26340 Austin English <austinenglish(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |download, source -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=26340 André H. <nerv(a)dawncrow.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |NEW CC| |nerv(a)dawncrow.de Ever Confirmed|0 |1 --- Comment #5 from André H. <nerv(a)dawncrow.de> 2011-03-08 13:21:49 CST --- confirming -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=26340 --- Comment #6 from ocean04(a)suomi24.fi 2011-06-13 09:00:05 CDT --- Now there are no other urlmon/wininet issues in 1.3.22. Winetricks msxml6 and example works! -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=26340 --- Comment #7 from ocean04(a)suomi24.fi 2011-06-27 04:40:56 CDT --- I added support for VT_VARIANT|VT_BYREF and example is now working. else if (V_VT(body) == VT_VARIANT|VT_BYREF) { HRESULT hrs; VARIANTARG tmpdat; VariantInit(&tmpdat); char *ansistr; hrs = VariantCopyInd(&tmpdat, body); if(hrs != S_OK) ERR("Could not copy array \n"); SAFEARRAY *sa = V_ARRAY(&tmpdat); LONG size = sa->rgsabound[0].cElements; TRACE("Got size %d\n", size); void *ptr; SafeArrayAccessData(sa, (void**)&ansistr); bsc->body = GlobalAlloc(GMEM_FIXED, size); if (!bsc->body) { heap_free(bsc); return E_OUTOFMEMORY; } ptr = GlobalLock(bsc->body); memcpy(ptr, ansistr, size); GlobalUnlock(bsc->body); SafeArrayUnaccessData(sa); } else FIXME("unsupported body data type %d\n", V_VT(body)); -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=26340 --- Comment #8 from Austin English <austinenglish(a)gmail.com> 2011-06-27 12:24:34 CDT --- (In reply to comment #7)
I added support for VT_VARIANT|VT_BYREF and example is now working.
else if (V_VT(body) == VT_VARIANT|VT_BYREF) { HRESULT hrs; VARIANTARG tmpdat; VariantInit(&tmpdat); char *ansistr; hrs = VariantCopyInd(&tmpdat, body); if(hrs != S_OK) ERR("Could not copy array \n"); SAFEARRAY *sa = V_ARRAY(&tmpdat); LONG size = sa->rgsabound[0].cElements; TRACE("Got size %d\n", size); void *ptr; SafeArrayAccessData(sa, (void**)&ansistr); bsc->body = GlobalAlloc(GMEM_FIXED, size); if (!bsc->body) { heap_free(bsc); return E_OUTOFMEMORY; } ptr = GlobalLock(bsc->body); memcpy(ptr, ansistr, size); GlobalUnlock(bsc->body); SafeArrayUnaccessData(sa); } else FIXME("unsupported body data type %d\n", V_VT(body));
Please send a patch to wine-patches(a)winehq.org. See http://wiki.winehq.org/SubmittingPatches for more info. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=26340 ocean04(a)suomi24.fi changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |patch --- Comment #9 from ocean04(a)suomi24.fi 2011-06-28 06:52:31 CDT --- I was hoping NS to do those tests.. I'm no C programmer and I guess it would not be accepted as it is. (There are compiler warnings) I was trying to access "body" directly, but it just did not work? Anyway, feel free to modify and send! Same goes for bug 27536 and bug 26355 -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=26340 --- Comment #10 from ocean04(a)suomi24.fi 2011-11-01 05:08:46 CDT --- Created attachment 37232 --> http://bugs.winehq.org/attachment.cgi?id=37232 Patch without compiler warnings -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=26340 Nikolay Sivov <bunglehead(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|wine-bugs(a)winehq.org |bunglehead(a)gmail.com --- Comment #11 from Nikolay Sivov <bunglehead(a)gmail.com> 2011-11-16 12:46:00 CST --- I'll try to find time for it this week. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=26340 --- Comment #12 from Nikolay Sivov <bunglehead(a)gmail.com> 2011-12-19 13:47:12 CST --- First part is fixed with 466341b6efd6ec9a6213df1081e232482fbde1e4. I have another patch to support inner variant type that is referred. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=26340 --- Comment #13 from Nikolay Sivov <bunglehead(a)gmail.com> 2011-12-20 14:02:40 CST --- Should be fixed with ea75ac8406e71f6cdb3ecfa45408bba5aebdd13e. Retest please. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=26340 ocean04(a)suomi24.fi changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords|download | Status|NEW |RESOLVED URL|http://netikka.net/dev/uplo | |ad.exe | Resolution| |FIXED --- Comment #14 from ocean04(a)suomi24.fi 2011-12-20 14:31:21 CST --- Post is now working here, many thanks! (Download sample was obsolete, because post url has changed) But seems this has caused regression to that other bug 27536 Can you check 27536 please? -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=26340 --- Comment #15 from Nikolay Sivov <bunglehead(a)gmail.com> 2011-12-20 20:55:58 CST --- (In reply to comment #14)
But seems this has caused regression to that other bug 27536
Can you check 27536 please?
http://bugs.winehq.org/show_bug.cgi?id=29395#c1 -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=26340 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #16 from Alexandre Julliard <julliard(a)winehq.org> 2011-12-30 12:57:13 CST --- Closing bugs fixed in 1.3.36. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=26340 Nikolay Sivov <bunglehead(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Fixed by SHA1| |ea75ac8406e71f6cdb3ecfa4540 | |8bba5aebdd13e -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=26340 Nikolay Sivov <bunglehead(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|bunglehead(a)gmail.com |wine-bugs(a)winehq.org -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
participants (1)
-
wine-bugs＠winehq.org