From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail.windriver.com (mail.windriver.com [147.11.1.11]) by mail.openembedded.org (Postfix) with ESMTP id 074346C7FB for ; Tue, 17 Sep 2013 11:10:27 +0000 (UTC) Received: from ALA-HCA.corp.ad.wrs.com (ala-hca.corp.ad.wrs.com [147.11.189.40]) by mail.windriver.com (8.14.5/8.14.3) with ESMTP id r8HBAQhv020922 (version=TLSv1/SSLv3 cipher=AES128-SHA bits=128 verify=FAIL); Tue, 17 Sep 2013 04:10:26 -0700 (PDT) Received: from [128.224.162.194] (128.224.162.194) by ALA-HCA.corp.ad.wrs.com (147.11.189.40) with Microsoft SMTP Server id 14.2.347.0; Tue, 17 Sep 2013 04:10:26 -0700 Message-ID: <5238389F.1080201@windriver.com> Date: Tue, 17 Sep 2013 19:10:23 +0800 From: Hongxu Jia User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130803 Thunderbird/17.0.8 MIME-Version: 1.0 To: "Burton, Ross" References: <5237C039.4080001@windriver.com> In-Reply-To: Cc: OE-core Subject: Re: [PATCH 1/1] libxml2: fix LSB desktop-xml tests failure X-BeenThere: openembedded-core@lists.openembedded.org X-Mailman-Version: 2.1.12 Precedence: list List-Id: Patches and discussions about the oe-core layer List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 17 Sep 2013 11:10:28 -0000 Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: 7bit On 09/17/2013 05:15 PM, Burton, Ross wrote: > On 17 September 2013 03:36, Hongxu Jia wrote: >> 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. > Using my amazing ability of talking to the upstream maintainer (DV in > #xml on irc.gnome.org) I've sorted this out. > > The CVE is for *Chromium's fork of libxml*. Not upstream libxml2. > The patch changes a public structure by adding fields *in the middle*, > so that broke the ABI. That's two good reasons to revert the patch. > As Daniel has said in the bug, 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. > > So, NAK to this patch, and a revert incoming. Great, the libxml2-CVE-2012-2871.patch is obsolete, abandon it could fix the LSB desktop-xml tests failure. I wll resend the patch to do this. Thanks, Hongxu > Ross