[Bug 20861] New: CoTaskMemAlloc()/SysFreeString() mismatch in DocHost_Release(), caught by valgrind in shdocvw/tests/webbrowser.c

http://bugs.winehq.org/show_bug.cgi?id=20861

           Summary: CoTaskMemAlloc()/SysFreeString() mismatch in
                    DocHost_Release(), caught by valgrind in
                    shdocvw/tests/webbrowser.c
           Product: Wine
           Version: 1.1.33
          Platform: PC
        OS/Version: Linux
            Status: NEW
          Keywords: download, source, testcase
          Severity: normal
          Priority: P2
         Component: urlmon
        AssignedTo: wine-bugs(a)winehq.org
        ReportedBy: dank(a)kegel.com


http://kegel.com/wine/valgrind/logs/2009-11-27-12.53/vg-shdocvw_webbrowser.t...
says
 Invalid free() / delete / delete[]
    at  notify_free (heap.c:255)
    by  RtlFreeHeap (heap.c:1748)
    by  SysFreeString (oleaut.c:194)
    by  DocHost_Release (dochost.c:756)
    by  WebBrowser_Release (webbrowser.c:159)
    by  test_WebBrowser (webbrowser.c:2367)
  Address 0x7f0ae944 is 4 bytes before a block of size 24 alloc'd
    at  notify_alloc (heap.c:247)
    by  RtlAllocateHeap (heap.c:1697)
    by  IMalloc_fnAlloc (ifs.c:186)
    by  CoTaskMemAlloc (ifs.c:562)
    by  URLMoniker_GetDisplayName (umon.c:383)

And sure enough, it looks like a mismatch between
CoTaskMemAlloc() and SysFreeString(), which sounds bad.

-- 
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.

[Bug 20861] CoTaskMemAlloc()/SysFreeString() mismatch in DocHost_Release(), caught by valgrind in shdocvw/tests/webbrowser.c

http://bugs.winehq.org/show_bug.cgi?id=20861


Jacek Caban <jacek(a)codeweavers.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jacek(a)codeweavers.com




--- Comment #1 from Jacek Caban <jacek(a)codeweavers.com>  2009-11-29 17:24:05 ---
Patch sent:
http://www.winehq.org/pipermail/wine-patches/2009-November/081914.html

-- 
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.

[Bug 20861] CoTaskMemAlloc()/SysFreeString() mismatch in DocHost_Release(), caught by valgrind in shdocvw/tests/webbrowser.c

http://bugs.winehq.org/show_bug.cgi?id=20861


Jacek Caban <jacek(a)codeweavers.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED




--- Comment #2 from Jacek Caban <jacek(a)codeweavers.com>  2009-12-01 06:39:41 ---
Fixed in Git.

-- 
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.

[Bug 20861] CoTaskMemAlloc()/SysFreeString() mismatch in DocHost_Release(), caught by valgrind in shdocvw/tests/webbrowser.c

http://bugs.winehq.org/show_bug.cgi?id=20861


Alexandre Julliard <julliard(a)winehq.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |CLOSED




--- Comment #3 from Alexandre Julliard <julliard(a)winehq.org>  2009-12-04 12:16:41 ---
Closing bugs fixed in 1.1.34.

-- 
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.