Re: mshtml #3: Store HTMLDocument pointer in BSCallback object.

[Alexandre Julliard]

Jacek Caban <jacek(a)codeweavers.com> writes:
> ---
>  dlls/mshtml/mshtml_private.h |    5 ++++-
>  dlls/mshtml/navigate.c       |   27 ++++++++++++++++++++++++---
>  dlls/mshtml/persist.c        |    2 +-
>  dlls/mshtml/task.c           |    7 ++++---
>  4 files changed, 33 insertions(+), 8 deletions(-)

That one is broken here:

../../../tools/runtest -q -P wine -M mshtml.dll -T ../../.. -p mshtml_test.exe.so dom.c && touch dom.ok
wine: Unhandled page fault on write access to 0x006d005d at address 0x61dfd127 (thread 0009), starting debugger...
WineDbg starting on pid 0008
Unhandled exception: page fault on write access to 0x006d005d in 32-bit code (0x61dfd127).
Register dump:
 CS:0023 SS:002b DS:002b ES:002b FS:0063 GS:006b
 EIP:61dfd127 ESP:0033fc30 EBP:0033fc48 EFLAGS:00010206(   - 00      - RIP1)
 EAX:00129c78 EBX:61e34074 ECX:006d0065 EDX:00129c78
 ESI:006d0025 EDI:00129cf4
Stack dump:
0x0033fc30:  00ee0314 00f84550 00000000 00000000
0x0033fc40:  804b0000 00129c78 0033fda8 61e123ea
0x0033fc50:  00129c78 00f96da8 00000000 00000000
0x0033fc60:  00000000 00000000 0033fc88 7bc4125e
0x0033fc70:  00129c78 61e2de2a 00ee0000 7bc3283f
0x0033fc80:  00000000 00f84538 0033fd38 00f96da8
Backtrace:
=>1 0x61dfd127 set_document_bscallback+0x57(doc=0x129c78, callback=0xf96da8) [/home/julliard/wine/wine/dlls/mshtml/navigate.c:855] in mshtml (0x0033fc48)
  2 0x61e123ea set_moniker+0x69a(This=0x129c78, mon=0xf84538, pibc=0x0, bind_complete=0x0) [/home/julliard/wine/wine/dlls/mshtml/persist.c:327] in mshtml (0x0033fda8)
  3 0x61e12a26 PersistStreamInit_Load+0x66(iface=0x129cbc, pStm=0xf84510) [/home/julliard/wine/wine/dlls/mshtml/persist.c:676] in mshtml (0x0033fde8)
  4 0x60512eb1 run_domtest+0x131(str=0x605284c0, test=0x605142b0) [/home/julliard/wine/wine/dlls/mshtml/tests/dom.c:1161] in mshtml_test (0x0033fe58)
  5 0x6051326c func_dom+0x3c() [/home/julliard/wine/wine/dlls/mshtml/tests/dom.c:1254] in mshtml_test (0x0033fe68)
  6 0x6052713a run_test+0x14a(name=0x1103e7) [/home/julliard/wine/wine/dlls/mshtml/tests/../../../include/wine/test.h:389] in mshtml_test (0x0033fea8)
  7 0x6052779a main+0x15a(argc=<register ECX not in topmost frame>, argv=0x1103c8) [/home/julliard/wine/wine/dlls/mshtml/tests/../../../include/wine/test.h:437] in mshtml_test (0x0033fed8)
  8 0x6052786b __wine_spec_exe_entry+0x5b(peb=0x7ffdf000) [/home/julliard/wine/wine/dlls/winecrt0/exe_entry.c:36] in mshtml_test (0x0033ff08)
  9 0x7b873dae start_process+0xee(arg=0x0) [/home/julliard/wine/wine/dlls/kernel32/process.c:839] in kernel32 (0x0033ffe8)
  10 0x600288a7 wine_switch_to_stack+0x17() in libwine.so.1 (0x00000000)
0x61dfd127 set_document_bscallback+0x57 [/home/julliard/wine/wine/dlls/mshtml/navigate.c:855] in mshtml: movl	$0x0,0x38(%esi)
855	        iter->doc = NULL;

-- 
Alexandre Julliard
julliard(a)winehq.org