user32/tests/dde.c: loosen timeout, fix allocation size

Dan Kegel dank at kegel.com
Wed Nov 14 23:12:37 CST 2007


Under valgrind, tests run a bit slower, so the 200ms
timeout seems to be too short; I saw this error in this morning's run:
dde.c:391: Test failed:  Expected TRUE, got 0x474197d
This patch doubles the timeout (though perhaps it should be bigger still).

Also, one test buffer was sized two bytes too big, which
caused a valgrind warning about the last two bytes being
uninitialised:
 Syscall param writev(vector[1]) points to uninitialised byte(s)
 ...by  test_msg_client (dde.c:1072)
    by  func_dde (dde.c:2080)
  Address 0x7F00A6F8 is 16 bytes inside a block of size 18 alloc'd
 ...by  create_poke (dde.c:982)
    by  test_msg_client (dde.c:1062)
    by  func_dde (dde.c:2080)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: user32-dde.patch
Type: text/x-patch
Size: 867 bytes
Desc: not available
Url : http://www.winehq.org/pipermail/wine-patches/attachments/20071114/f8ab56d7/attachment-0001.bin 


More information about the wine-patches mailing list