Bug 5151

Summary: LSB Desktop Tests: desktop-xml failed
Product: [Build System, Metadata & Runtime] OE-Core Reporter: Hongxu Jia <hongxu.jia>
Component: coreAssignee: Hongxu Jia <hongxu.jia>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: Medium CC: hongxu.jia, liezhi.yang, mark.hatle, meta.mr.watcher, meta.watcher, sgw, yi.zhao
Version: 1.5   
Target Milestone: 1.5 M5   
Hardware: x86   
OS: Multiple   
Whiteboard:
OS type for building Yocto: --- Type of Regression: ---
Verified: Documentation change: Don't know

Description Hongxu Jia 2013-09-10 04:51:53 UTC 
LSB Desktop Tests: desktop-xml failed
...
XML_30 6 failed 
XML_31 6 failed 
XML_31 7 failed 
XML_31 8 failed 
XML_31 9 failed 
XML_31 10 failed 
XML_31 11 failed 
XML_31 12 failed 
XML_31 13 failed 
XML_31 14 failed 
XML_31 15 failed 
XML_31 17 failed 
XML_31 18 failed 
XML_32 1 failed 
XML_32 2 failed 
XML_32 3 failed 
XML_32 4 failed 
XML_32 5 failed 
XML_32 6 failed 
XML_32 7 failed 
XML_32 8 failed 
XML_32 9 failed 
XML_32 10 failed 
XML_32 11 failed 
XML_32 12 failed 
XML_32 13 failed 
XML_32 14 failed 
XML_32 15 failed 
XML_32 16 failed 
XML_32 17 failed 
XML_32 18 failed 
...

Each failed has the same journal:
...
220|31 6 1 16:02:49|FAIL
410|31 6 1 16:02:49|IC End
400|31 7 1 16:02:49|IC Start
200|31 7 16:02:49|./test/c14n/without-comments/merlin-c14n-two-00.xml
520|31 7 0 0 0|Error: unable to register NS with prefix="(null)" and href="http://example.org/bar"
520|31 7 0 0 0|Error: unable to evaluate xpath expression
520|31 7 0 0 0|File ./test/c14n/without-comments/merlin-c14n-two-00.xml generated an error
520|31 7 0 0 0|
...
Comment 1 Hongxu Jia 2013-09-16 11:06:35 UTC 
The commit
http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/?id=8780c5ddf2916bbd42fc67b79c286652aebb1546
add a patch to fix a security issue. It modify include file 'tree.h'
to add 'const char *dummy_children' for 'struct _xmlNs'.

But lsb test suites didn't do this in his own include file, so the LSB
desktop-xml tests failed.

Disable this patch for linuxstdbase could fix this issue.
Comment 2 Hongxu Jia 2013-09-17 03:07:28 UTC 
Hi Saul,

The upstream of libxml2 has not fixed this issue:
git clone git://git.gnome.org/libxml2

And I have filed a bug to them
https://bugzilla.gnome.org/show_bug.cgi?id=708205

After this is fixed and released, also need to report another
bug to LSB to update their libxml2 source code.

The time cycle is long, should we mark this bug as "Waiting For Upstream"
or accept this patch to workaround for LSB test.

Thanks,
Hongxu
Comment 3 Hongxu Jia 2013-09-17 11:19:32 UTC 
The libxml2-CVE-2012-2871.patch changes a public structure by adding
fields *in the middle*, so that broke the ABI. This patch was the quick
fix that Chromium did as they statically link to libxml2 so the API
breakage isn't an issue, the proper fix is already in libxslt.  As long
as we have libxml 2.9.0 and libxslt 1.1.27 onwards (which we do), the
issue is correctly fixed. 

The patch is obsolete, abandon it could fix the LSB desktop-xml tests
failure.

Ross has done the revert job.