[Bug 30505] New: Microsoft Office Communicator 2007 R2 Trial fails to parse rule XML from resource file (needs support for push/pop operations for namespace contexts)

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

Bug #: 30505
           Summary: Microsoft Office Communicator 2007 R2 Trial fails to
                    parse rule XML from resource file (needs support for
                    push/pop operations for namespace contexts)
           Product: Wine
           Version: 1.5.2
          Platform: x86
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: P2
         Component: msxml3
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]
    Classification: Unclassified

Hello,

continuation of bug 30491 (needs
http://bugs.winehq.org/attachment.cgi?id=39876)

Prerequisite: 'winetricks win7 vcrun2008'

After installation (one liner):

--- snip ---
$ wine reg add "HKLM\Software\Microsoft\Communicator\3.5\Registration" /v
"ProductID" /t "REG_SZ" /d "01234567890123456789012" 
--- snip ---

Relevant trace log:

--- snip ---
$ WINEDEBUG=+tid,+msxml wine ./communicator.exe
...
0009:trace:msxml:DllGetClassObject {88d96a11-f192-11d4-a65f-0040963251e5}
{00000001-0000-0000-c000-000000000046} 0x32e838
0009:trace:msxml:ClassFactory_CreateInstance (nil)
{c90352f5-643c-4fbc-bb23-e996eb2d51fd} 0x32e8a4
0009:trace:msxml:MXNamespaceManager_create ((nil), 0x32e7d8)
0009:trace:msxml:MXNamespaceManager_create returning iface 0x1949b8
0009:trace:msxml:vbnamespacemanager_QueryInterface
(0x1949a8)->({c90352f5-643c-4fbc-bb23-e996eb2d51fd} 0x32e8a4)
0009:trace:msxml:vbnamespacemanager_AddRef (0x1949a8)->(2)
0009:trace:msxml:vbnamespacemanager_Release (0x1949a8)->(1)
0009:trace:msxml:vbnamespacemanager_put_allowOverride (0x1949a8)->(1)
0009:trace:msxml:vbnamespacemanager_AddRef (0x1949a8)->(2)
0009:trace:msxml:vbnamespacemanager_Release (0x1949a8)->(1)
0009:trace:msxml:internal_parse (0x13ae88)->({VT_BSTR:
L"<categoryPublicationManifest\r\n 
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance%5C%22%5Cr%5Cn 
xmlns:xsd="http://www.w3.org/2001/XMLSchema%5C%22%5Cr%5Cn 
xmlns="http://schemas.microsoft.com/200)
0009:trace:msxml:saxxmlreader_AddRef 0x13ae88
0009:trace:msxml:SAXLocator_create returning 0x194ad0
0009:trace:msxml:internal_parseBuffer detected encoding: (null)
0009:trace:msxml:internal_parseBuffer switching to UTF-16
0009:trace:msxml:isaxlocator_AddRef (0x194ad0)->(2)
0009:fixme:msxml:vbnamespacemanager_pushContext (0x1949a8): stub
0009:fixme:msxml:format_error_message_from_id MSXML errors not yet supported.
0009:trace:msxml:isaxlocator_Release (0x194ad0)->(1)
0009:trace:msxml:vbnamespacemanager_Release (0x1949a8)->(0)
0009:trace:msxml:internal_putContentHandler (0x13ae88)->((nil))
0009:trace:msxml:internal_putProperty
(0x13ae88)->(L"http://xml.org/sax/properties/lexical-handler" {VT_UNKNOWN:
(nil)})
0009:trace:msxml:internal_putErrorHandler (0x13ae88)->((nil))
0009:trace:msxml:saxxmlreader_Release 0x13ae88
0009:trace:msxml:isaxlocator_Release (0x194ad0)->(0)
0009:trace:msxml:saxxmlreader_Release 0x13ae88
--- snip ---

$ du -sh CommunicatorEval.msi 
15M    CommunicatorEval.msi

$ sha1sum CommunicatorEval.msi 
b2a1cc95362b93aef6dfbf9987cc5c4ca6c8038d  CommunicatorEval.msi

$ wine --version
wine-1.5.2-191-gd080774

Regards

[Bug 30505] Microsoft Office Communicator 2007 R2 Trial fails to parse rule XML from resource file (needs support for push/pop operations for namespace contexts)

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

Anastasius Focht  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |download
                URL|                            |http://www.microsoft.com/do
                   |                            |wnload/en/details.aspx?disp
                   |                            |laylang=en&id=8468
         Depends on|                            |30491

--- Comment #1 from Anastasius Focht  2012-04-22 15:20:52 CDT ---
Hello,

filling fields.

Regards

[Bug 30505] Microsoft Office Communicator 2007 R2 Trial fails to parse rule XML from resource file (needs support for push/pop operations for namespace contexts)

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

--- Comment #2 from Anastasius Focht  2012-04-22 15:25:05 CDT ---
Created attachment 39891
  --> http://bugs.winehq.org/attachment.cgi?id=39891
WINEDEBUG=+tid,+seh,+relay,+msxml wine ./communicator.exe >>log.txt 2>&1

Hello,

the patch from http://bugs.winehq.org/show_bug.cgi?id=30491#c4
(http://bugs.winehq.org/attachment.cgi?id=39890) lets it get further.

It now seems to crash in CDATA related events handling.

--- snip ---
...
Unhandled exception: page fault on read access to 0x435b214c in 32-bit code
(0x21fbc538).
Register dump:
 CS:0023 SS:002b DS:002b ES:002b FS:0063 GS:006b
 EIP:21fbc538 ESP:0033e408 EBP:0033e420 EFLAGS:00010286(  R- --  I S - -P- )
 EAX:435b213c EBX:00138600 ECX:0033e42c EDX:f7460a97
 ESI:21cc4838 EDI:7d68110b
Stack dump:
0x0033e408:  7d68110b 0033e42c 0013862c 21cc4838
0x0033e418:  222e5038 8000ffff 0033e444 21fbd7d2
0x0033e428:  ffffffff ffffffff 7d9987a8 7d997460
0x0033e438:  7d6a5260 0013862c 0013862c 0033e494
0x0033e448:  7d65777f 00138600 7d68110b 00000009
0x0033e458:  f7460a97 00000648 00000c88 0033e4a4
Backtrace:
=>0 0x21fbc538 in ocapi (+0x33c538) (0x0033e420)
  1 0x21fbd7d2 in ocapi (+0x33d7d1) (0x0033e444)
  2 0x7d65777f libxmlCDataBlock+0x16f(ctx=0x192e48, value="
<xsl:stylesheet
   xmlns:xsl="http://www.w3.org/1999/XSL/Transform"
   version="1.0"
   xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
   xmlns:ct="http://schemas.microsoft.com/2006/09/sip/commontypes"
   xmlns:tns="http://schemas.microsoft.com/2006/09/sip/state"
   exclude-result-prefixes="tns ct">
      <xsl:output method="xml" encoding="utf-8" indent="yes"/>
      <xsl:strip-space elements="*"/>
      <xsl:template match="node()">
         <xsl:apply-templates select="//tns:state"/>
      </xsl:template>
      <xsl:template match="tns:state">
         xsl:copy
            <xsl:copy-of select="@*" />
            xsl:choose
               <xsl:when test="@xsi:type='userState' and tns:availability>=9000
and 11999>=tns:availability and @manual='true'">
                  <xsl:element name="availability"
xmlns="http://schemas.microsoft.com/2006/09/sip/state%22%3E6900</xsl:element>
                  <xsl:element name="activity"
xmlns="http://schemas.microsoft.com/2006/09/sip/state%22%3E
                     ", len=0xa4b)
[/home/focht/projects/wine/wine-git/dlls/msxml3/saxreader.c:1688] in msxml3
(0x0033e494)
  3 0x7d4c0119 xmlParseCDSect+0x4e8() in libxml2.so.2 (0x00000a4b)
  4 0x7d4d0f78 xmlParseContent+0x1f7() in libxml2.so.2 (0x00000000)
  5 0x7d4d2044 xmlParseElement+0x143() in libxml2.so.2 (0x00000006)
  6 0x7d4d0ec0 xmlParseContent+0x13f() in libxml2.so.2 (0x00000000)
  7 0x7d4d2044 xmlParseElement+0x143() in libxml2.so.2 (0x00000006)
  8 0x7d4d0ec0 xmlParseContent+0x13f() in libxml2.so.2 (0x00000000)
  9 0x7d4d2044 xmlParseElement+0x143() in libxml2.so.2 (0x00000006)
  10 0x7d4d0ec0 xmlParseContent+0x13f() in libxml2.so.2 (0x00000000)
  11 0x7d4d2044 xmlParseElement+0x143() in libxml2.so.2 (0x00000000)
  12 0x7d4d3072 xmlParseDocument+0x251() in libxml2.so.2 (0x0033e7c4)
--- snip ---

Attached is WINEDEBUG=+tid,+seh,+relay,+msxml log (uncompressed: 10 MiB).

Regards

[Bug 30505] Microsoft Office Communicator 2007 R2 Trial fails to parse rule XML from resource file (needs support for push/pop operations for namespace contexts)

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

Bug 30505 depends on bug 30491, which changed state.

Bug 30491 Summary: Microsoft Office Communicator 2007 R2 Trial fails to parse rule XML from resource file (BSTR/UTF-16 passed to SAXXMLReader)
http://bugs.winehq.org/show_bug.cgi?id=30491

What    |Old Value                   |New Value
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED

[Bug 30505] Microsoft Office Communicator 2007 R2 Trial fails to parse rule XML from resource file (needs support for push/pop operations for namespace contexts)

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

--- Comment #3 from Nikolay Sivov  2012-04-23 13:57:26 CDT ---
Stub from initial log is implemented by
43241140aaa1bf0f7cee277488e599ae721d4d59.

[Bug 30505] Microsoft Office Communicator 2007 R2 Trial fails to parse rule XML from resource file (needs support for push/pop operations for namespace contexts)

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

--- Comment #4 from Anastasius Focht  2012-04-24 02:34:39 CDT ---
Hello Nikolay,

--- quote ---
Stub from initial log is implemented by
43241140aaa1bf0f7cee277488e599ae721d4d59.
--- quote ---

thanks for the work.
Do you think the crash within CDATA event handling deserves a new bug?

Regards

[Bug 30505] Microsoft Office Communicator 2007 R2 Trial fails to parse rule XML from resource file (needs support for push/pop operations for namespace contexts)

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

--- Comment #5 from Nikolay Sivov  2012-04-24 03:16:11 CDT ---
(In reply to comment #4)
> Do you think the crash within CDATA event handling deserves a new bug?

Yes, please. It looks unrelated, I suspect two things so far - we mess with
input stream a bit before calling handlers, could be buggy. Second thought is
used chunk length for characters() callback, and there's no tests for CDATA
section for a reader apparently.

[Bug 30505] Microsoft Office Communicator 2007 R2 Trial fails to parse rule XML from resource file (needs support for push/pop operations for namespace contexts)

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

Anastasius Focht  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
      Fixed by SHA1|                            |43241140aaa1bf0f7cee277488e
                   |                            |599ae721d4d59
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED

--- Comment #6 from Anastasius Focht  2012-04-24 13:43:44 CDT ---
Hello Nikolay,

thanks, marking it fixed by commit
http://source.winehq.org/git/wine.git/commitdiff/43241140aaa1bf0f7cee277488e...

I'll file a new bug for the current crash.

Regards

[Bug 30505] Microsoft Office Communicator 2007 R2 Trial fails to parse rule XML from resource file (needs support for push/pop operations for namespace contexts)

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

Alexandre Julliard  changed:

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

--- Comment #7 from Alexandre Julliard  2012-04-27 13:00:00 CDT ---
Closing bugs fixed in 1.5.3.

[Bug 30505] Microsoft Office Communicator 2007 R2 Trial fails to parse rule XML from resource file (needs support for push/pop operations for namespace contexts)

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=30505

Anastasius Focht  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                URL|http://www.microsoft.com/do |https://web.archive.org/web
                   |wnload/en/details.aspx?disp |/20130312155123/http://down
                   |laylang=en&id=8468          |load.microsoft.com/download
                   |                            |/E/2/B/E2B88A20-CC84-4F29-8
                   |                            |C15-751ADFB8F0BF/Communicat
                   |                            |orEval.msi

--- Comment #8 from Anastasius Focht  ---
Hello folks,

adding stable download link via Internet Archive for documentation.

https://web.archive.org/web/20130312155123/http://download.microsoft.com/dow...

https://www.virustotal.com/gui/file/8b5a177b9367101e262f0e31bc3685e07a898b84...

$ sha1sum CommunicatorEval.msi 
b2a1cc95362b93aef6dfbf9987cc5c4ca6c8038d  CommunicatorEval.msi

$ du -sh CommunicatorEval.msi 
15M    CommunicatorEval.msi

Regards