Bug 1637667

Summary: osinfo-db: enforce anchored regexes
Product: [Community] Virtualization Tools Reporter: Cole Robinson <crobinso>
Component: libosinfoAssignee: Fabiano FidĂȘncio <fidencio>
Status: CLOSED DEFERRED QA Contact:
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: unspecifiedCC: berrange, cfergeau, crobinso, fidencio
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2019-03-29 00:01:13 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Cole Robinson 2018-10-09 17:39:46 UTC
Fabiano switched libosinfo to assume regexs as anchored: https://www.redhat.com/archives/libosinfo/2018-September/msg00060.html

Dan disagreed with this approach: https://www.redhat.com/archives/libosinfo/2018-October/msg00033.html

Fabiano reverted: https://www.redhat.com/archives/libosinfo/2018-October/msg00038.html

However I think we should consider meeting the same goal, and explicitly anchor all regexs and either enforce this with a schema change or add a test case to catch it. Non-anchored regex can cause subtle issues like were pointed out in the original series

Comment 1 Cole Robinson 2019-03-29 00:01:13 UTC
I moved this to the gitlab tracker: https://gitlab.com/libosinfo/osinfo-db/issues/23