[Bug 26053] New: urlmon/sec_mgr tests show some invalid writes under valgrind

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

Summary: urlmon/sec_mgr tests show some invalid writes under
                    valgrind
           Product: Wine
           Version: 1.3.13
          Platform: x86
        OS/Version: Linux
            Status: NEW
          Keywords: download, source, testcase
          Severity: minor
          Priority: P2
         Component: urlmon
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]

Created an attachment (id=33225)
 --> (http://bugs.winehq.org/attachment.cgi?id=33225)
valgrind log

Invalid write of size 4
    at  memcpy (mc_replace_strmem.c:635)
    by  InternetProtocolInfo_ParseUrl (sec_mgr.c:873)
    by  parse_security_url (sec_mgr.c:1270)
    by  CoInternetGetSecurityUrl (sec_mgr.c:1372)
    by  test_InternetGetSecurityUrl (sec_mgr.c:787)
    by  func_sec_mgr (sec_mgr.c:1188)
    by  run_test (test.h:556)
    by  main (test.h:624)
  Address 0x7f01fe30 is 10 bytes after a block of size 14 free'd
    at  notify_free (heap.c:262)
    by  RtlFreeHeap (heap.c:1747)
    by  RtlFreeUnicodeString (rtlstr.c:319)
    by  GetPrivateProfileStringA (profile.c:1185)
    by  GetProfileStringA (profile.c:1199)
    by  SYSPARAMS_Init (sysparams.c:941)
    by  process_attach (user_main.c:276)
    by  DllMain (user_main.c:332)
    by  __wine_spec_dll_entry (dll_entry.c:40)
    by  ??? (loader.c:139)
    by  MODULE_InitDLL (loader.c:978)
    by  process_attach (loader.c:1067)
    by  process_attach (loader.c:1059)
    by  process_attach (loader.c:1059)
    by  process_attach (loader.c:1059)
    by  attach_process_dlls (loader.c:2535)
    by  ??? (port.c:60)

[Bug 26053] urlmon/sec_mgr tests show some invalid writes under valgrind

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

--- Comment #1 from Austin English  2011-02-14 16:27:45 CST ---
Improved by:
http://source.winehq.org/git/wine.git/?a=commitdiff;h=9896d41ced9fd2b1904cfa...

but still some leaks. I'll attach today's log.

[Bug 26053] urlmon/sec_mgr tests show some invalid writes under valgrind

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

Austin English  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #33225|0                           |1
        is obsolete|                            |

--- Comment #2 from Austin English  2011-02-14 16:27:58 CST ---
Created an attachment (id=33287)
 --> (http://bugs.winehq.org/attachment.cgi?id=33287)
valgrind log

[Bug 26053] urlmon/sec_mgr tests show some invalid writes under valgrind

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

Dan Kegel  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
                 CC|                            |[e-mail]
         Resolution|                            |FIXED

--- Comment #3 from Dan Kegel  2011-09-24 10:27:32 CDT ---
Fixed, then.

In general, we should not consider conformance test memory leaks found by
valgrind as bugs unless they're O(n).

[Bug 26053] urlmon/sec_mgr tests show some invalid writes under valgrind

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

Alexandre Julliard  changed:

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

--- Comment #4 from Alexandre Julliard  2011-10-10 13:13:55 CDT ---
Closing bugs fixed in 1.3.30.