diff mbox series

[v2] cve-check: Log if CVE_STATUS set but not reported for component

Message ID 20240224081838.60542-1-simone.p.weiss@posteo.com
State Accepted, archived
Commit 013d531a84fa08b6ae8a47bdf3ba1fa8f18ba270
Headers show
Series [v2] cve-check: Log if CVE_STATUS set but not reported for component | expand

Commit Message

Simone Weiß Feb. 24, 2024, 8:18 a.m. UTC 
From: Simone Weiß <simone.p.weiss@posteo.com>

Log if the CVE_STATUS is set for a CVE, but the cve is not reported for a
component. This should hopefully help to clean up not needed CVE_STATUS
settings.

Signed-off-by: Simone Weiß <simone.p.weiss@posteo.com>
---
v2: Add this to oe.qa mechanism instead of using bb.warn. This way it will only
be executed if one adds it to WARN_QA. Not added there by default, so it should
not be bothering unless a explicit check is wanted.

 meta/classes/cve-check.bbclass | 3 +++
 1 file changed, 3 insertions(+)
diff mbox series

Patch

diff --git a/meta/classes/cve-check.bbclass b/meta/classes/cve-check.bbclass
index 5191d04303..56ba8bceef 100644
--- a/meta/classes/cve-check.bbclass
+++ b/meta/classes/cve-check.bbclass
@@ -418,6 +418,9 @@  def check_cves(d, patched_cves):
             cves_status.append([product, False])
 
     conn.close()
+    diff_ignore = list(set(cve_ignore) - set(cves_ignored))
+    if diff_ignore:
+        oe.qa.handle_error("cve_status_not_in_db", "Found CVE (%s) with CVE_STATUS set that are not found in database for this component" % " ".join(diff_ignore), d)
 
     if not cves_in_recipe:
         bb.note("No CVE records for products in recipe %s" % (pn))