Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
To: buildroot@busybox.net
Subject: [Buildroot] [NEXT 06/26] boa: add CPE id
Date: Wed, 28 Feb 2018 07:38:27 +0100	[thread overview]
Message-ID: <20180228073827.312a9a6f@windsurf.home> (raw)
In-Reply-To: <CANQCQpbM3VHid2rGTFAA=HwkhsUfmHRbGw9EDtT2dPUWwQRDog@mail.gmail.com>

Hello,

On Tue, 27 Feb 2018 22:00:31 -0600, Matthew Weber wrote:

> > LIBZLIB_CPE_ID_VENDOR = gnu
> > LIBZLIB_CPE_ID_NAME = zlib
> >  
> 
> Where it gets difficult is when a package has multiple CPEs so that
> you catch all possible combinations that could be relevant for the
> package.  Eventually there shouldn't be multiple once the dictionary
> at NIST is cleaned up......

Well, my proposal contained:

$(2)_CPE_ID ?= $($(2)_CPE_ID_VENDOR):$($(2)_CPE_ID_NAME):$($(2)_CPE_ID_VERSION)

Notice the ?=, which still allows a given package to override its
<pkg>_CPE_ID value entirely, potentially with a list of multiple values.

> One thing I didn't think about with the multiple CPEs is the logic to
> detect if a CPE needs updating in the CPE dictionary.  We'd have to
> look at all possible CPE id options to see if any match.  This may
> lead to us wanting to push to update the primary CPE ID sooner then
> later and align all our packages to use it instead of supporting
> multiple.  Then we could use the logic of having all packages set the
> LIBZLIB_CPE_ID_VENDOR and key off that to know we report that packages
> info.

I am not sure what is the story being multiple CPE IDs. Is it the NIST
database having two entries for what is essentially the same upstream
software ? If that's the case, then it really is a bug in the NIST
database, which should rather be fixed in the database rather than
worked around on our side by supporting CPE IDs, no ?

Of course, fixing the NIST database is the ideal situation, but it is
only reasonable if the NIST people are interested in fixing those
issues in a timely fashion. If they are not, then I'd say it's fine to
support multiple CPE IDs in Buildroot for the time being.

> > The drawback is obviously that all packages will suddenly have a
> > <pkg>_CPE_ID value, even if this value may potentially be incorrect
> > because it hasn't been checked. And this may be a real problem, so
> > probably your solution is better, I just wanted to point out the
> > (admittedly limited) duplication.  
> 
> Yeah, eventually I'd argue we could revert back to infra logic and
> clean it all up once there is a baseline or majority of packages with
> valid CPE IDs.    However for now, I default those we don't have to
> unknown so it's clear for a developer to go research or request a new
> CPE.

Yes, that is a possibility. As I said, I do realize that my proposal
has the significant drawback of not allowing to identify clearly which
package has a correct CPE_ID (voluntarily added by a developer) vs.
some potentially random CPE_ID (set by default by the package
infrastructure).

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
http://bootlin.com

  reply	other threads:[~2018-02-28  6:38 UTC|newest]

Thread overview: 52+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-02-27  2:10 [Buildroot] [NEXT 00/26] Package CVE Reporting Matt Weber
2018-02-27  2:10 ` [Buildroot] [NEXT 01/26] cpe-info: new make target Matt Weber
2018-02-27 21:40   ` Thomas Petazzoni
2018-02-28  4:30     ` Matthew Weber
2018-03-01 20:21       ` Arnout Vandecappelle
2018-02-27  2:10 ` [Buildroot] [NEXT 02/26] cpe-info: update manual for new pkg vars Matt Weber
2018-02-27 21:43   ` Thomas Petazzoni
2018-02-28  4:22     ` Matthew Weber
2018-02-27  2:10 ` [Buildroot] [NEXT 03/26] cpe-info: id prefix/suffix Matt Weber
2018-02-27 21:45   ` Thomas Petazzoni
2018-02-28  4:14     ` Matthew Weber
2018-03-01 20:34       ` Arnout Vandecappelle
2018-03-03  3:01         ` Matthew Weber
2018-03-01 20:32   ` Arnout Vandecappelle
2018-02-27  2:10 ` [Buildroot] [NEXT 04/26] cpe-info: only report target pkgs Matt Weber
2018-02-27 21:45   ` Thomas Petazzoni
2018-02-28  4:13     ` Matthew Weber
2018-02-27  2:10 ` [Buildroot] [NEXT 05/26] bash: add CPE id Matt Weber
2018-02-27  2:10 ` [Buildroot] [NEXT 06/26] boa: " Matt Weber
2018-02-27 22:17   ` Thomas Petazzoni
2018-02-28  4:00     ` Matthew Weber
2018-02-28  6:38       ` Thomas Petazzoni [this message]
2018-03-01 20:47         ` Arnout Vandecappelle
2018-03-01 22:55           ` Matthew Weber
2018-03-02  8:19             ` Arnout Vandecappelle
2018-03-02  9:49               ` Thomas Petazzoni
2018-03-02 16:14                 ` Matthew Weber
2018-02-27  2:10 ` [Buildroot] [NEXT 07/26] boost: " Matt Weber
2018-02-27  2:10 ` [Buildroot] [NEXT 08/26] busybox: " Matt Weber
2018-02-27  2:10 ` [Buildroot] [NEXT 09/26] bzip2: " Matt Weber
2018-02-27  2:10 ` [Buildroot] [NEXT 10/26] dhcp: " Matt Weber
2018-02-27  2:10 ` [Buildroot] [NEXT 11/26] e2fsprogs: " Matt Weber
2018-02-27  2:10 ` [Buildroot] [NEXT 12/26] gdb: " Matt Weber
2018-02-27  2:10 ` [Buildroot] [NEXT 13/26] glibc: " Matt Weber
2018-02-27  2:10 ` [Buildroot] [NEXT 14/26] gnupg: " Matt Weber
2018-02-27  2:10 ` [Buildroot] [NEXT 15/26] gzip: " Matt Weber
2018-02-27  2:10 ` [Buildroot] [NEXT 16/26] iproute2: " Matt Weber
2018-02-27  2:10 ` [Buildroot] [NEXT 17/26] libgcrypt: " Matt Weber
2018-02-27  2:10 ` [Buildroot] [NEXT 18/26] libopenssl: " Matt Weber
2018-02-27  2:10 ` [Buildroot] [NEXT 19/26] libzlib: " Matt Weber
2018-02-27  2:10 ` [Buildroot] [NEXT 20/26] linux: " Matt Weber
2018-02-27 22:18   ` Thomas Petazzoni
2018-02-28  4:12     ` Matthew Weber
2018-03-02  9:55       ` Thomas Petazzoni
2018-02-27  2:10 ` [Buildroot] [NEXT 21/26] linux-headers: " Matt Weber
2018-02-27  2:10 ` [Buildroot] [NEXT 22/26] openssh: " Matt Weber
2018-02-27  2:10 ` [Buildroot] [NEXT 23/26] rsyslog: " Matt Weber
2018-02-27  2:10 ` [Buildroot] [NEXT 24/26] tcpdump: " Matt Weber
2018-02-27  2:10 ` [Buildroot] [NEXT 25/26] util-linux: " Matt Weber
2018-02-27  2:10 ` [Buildroot] [NEXT 26/26] xerces: " Matt Weber
2018-02-27 21:37 ` [Buildroot] [NEXT 00/26] Package CVE Reporting Thomas Petazzoni
2018-02-28  4:42   ` Matthew Weber

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=20180228073827.312a9a6f@windsurf.home \
    --to=thomas.petazzoni@bootlin.com \
    --cc=buildroot@busybox.net \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox