All of lore.kernel.org
 help / color / mirror / Atom feed
From: Steve Sakoman <steve@sakoman.com>
To: Mike Crowe <mac@mcrowe.com>
Cc: Ralph Siemsen <ralph.siemsen@linaro.org>,
	Ross Burton <ross@burtonini.com>,
	"Mittal, Anuj" <anuj.mittal@intel.com>,
	Patches and discussions about the oe-core layer
	<openembedded-core@lists.openembedded.org>
Subject: Re: [OE-core] [PATCH][dunfell] zlib: backport the fix for CVE-2018-25032
Date: Wed, 13 Apr 2022 09:16:52 -1000	[thread overview]
Message-ID: <CAOSpxdYyFbSfZ7Jpz61AL_Nt1+Gg4j9eVjLK5+WgUFN_cXX-pg@mail.gmail.com> (raw)
In-Reply-To: <CAOSpxdbaE-=NOsQVG_+tK1J9oojA9qT8sMoR3c06uEssXJ0mFQ@mail.gmail.com>

On Wed, Apr 13, 2022 at 7:37 AM Steve Sakoman <steve@sakoman.com> wrote:
>
> On Wed, Apr 13, 2022 at 6:41 AM Mike Crowe <mac@mcrowe.com> wrote:
> >
> > On Wednesday 13 April 2022 at 06:02:22 -1000, Steve Sakoman wrote:
> > > Both runs completed and I'm still seeing success without the zlib patch:
> > >
> > > https://autobuilder.yoctoproject.org/typhoon/#/builders/50/builds/5069
> > >
> > > and failure with the patch:
> > >
> > > https://autobuilder.yoctoproject.org/typhoon/#/builders/50/builds/5070
> >
> > I'm certainly no expert with the autobuilder, but it looks like nothing was
> > actually compiled for both of those builds - everything came from the
> > sstate cache.
> >
> > I believe that Ralph's reproduction of the test failure without the zlib
> > patch was from a complete rebuild without anything coming from the sstate
> > cache.
> >
> > I suspect that if a PR bump or something similar that causes zlib and all
> > its reverse dependencies to be built were tested on top of the commit used
> > for build 5069 then the test failure would occur then as well and
> > exonerate the zlib patch.
>
> A valid point, let's see what happens with a PR bump:
>
> https://autobuilder.yoctoproject.org/typhoon/#/builders/50/builds/5072
>
> I see plenty of rebuilds in process . . .

As you predicted, rebuilding zlib (and all dependencies) with a PR
bump did indeed result in the same failure, exonerating the zlib CVE
patch.

So it really does appear that we are chasing a bug in the native
apt-ftparchive command on fedora-35 (and likely alma-8 since I've seen
the error there too)

Steve


  parent reply	other threads:[~2022-04-14 16:03 UTC|newest]

Thread overview: 38+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-03-29 13:07 [PATCH][dunfell] zlib: backport the fix for CVE-2018-25032 Ross Burton
2022-03-29 19:49 ` [OE-core] " Steve Sakoman
2022-03-30 20:40   ` Ross Burton
2022-03-30 22:57     ` Steve Sakoman
2022-04-04 15:22       ` Steve Sakoman
2022-04-11 17:31         ` Ralph Siemsen
2022-04-11 17:52           ` Steve Sakoman
2022-04-11 18:17             ` Ralph Siemsen
2022-04-11 18:58               ` Steve Sakoman
2022-04-11 22:30                 ` Ralph Siemsen
2022-04-11 22:42                   ` Steve Sakoman
2022-04-12  1:33                     ` Ralph Siemsen
2022-04-12  2:12                       ` Steve Sakoman
2022-04-12 16:07                         ` Ralph Siemsen
2022-04-12 21:49                           ` Steve Sakoman
2022-04-13  1:21                             ` Ralph Siemsen
2022-04-13 15:31                               ` Steve Sakoman
     [not found]                               ` <16E57E79FD292EFA.13992@lists.openembedded.org>
2022-04-13 16:02                                 ` Steve Sakoman
2022-04-13 16:41                                   ` Mike Crowe
2022-04-13 17:37                                     ` Steve Sakoman
2022-04-13 17:50                                       ` Jose Quaresma
2022-04-13 19:16                                       ` Steve Sakoman [this message]
2022-04-13 18:02                                     ` Ralph Siemsen
2022-04-13 18:19                                       ` Steve Sakoman
2022-04-13 19:05                                         ` Ralph Siemsen
2022-04-13 21:39                                           ` Steve Sakoman
2022-04-13 21:41                                             ` Richard Purdie
2022-04-13 21:44                                               ` Steve Sakoman
2022-04-14  2:47                                                 ` Ralph Siemsen
2022-04-14  3:00                                                   ` Steve Sakoman
     [not found]                                                   ` <16E5A41A6E4FF34A.8845@lists.openembedded.org>
2022-04-14  3:02                                                     ` Steve Sakoman
2022-04-14  3:03                                                       ` Steve Sakoman
2022-04-13 17:11                                   ` Jose Quaresma
     [not found]                                   ` <16E583EB139C493B.16998@lists.openembedded.org>
2022-04-13 17:41                                     ` Jose Quaresma
2022-03-31  0:24     ` Mittal, Anuj
2022-04-05 19:04 ` Jeroen Hofstee
2022-04-09 18:14   ` Steve Sakoman
2022-04-10 21:21     ` Jeroen Hofstee

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CAOSpxdYyFbSfZ7Jpz61AL_Nt1+Gg4j9eVjLK5+WgUFN_cXX-pg@mail.gmail.com \
    --to=steve@sakoman.com \
    --cc=anuj.mittal@intel.com \
    --cc=mac@mcrowe.com \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=ralph.siemsen@linaro.org \
    --cc=ross@burtonini.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.