Bug 1637667 - osinfo-db: enforce anchored regexes
Summary: osinfo-db: enforce anchored regexes
Keywords:
Status: CLOSED DEFERRED
Alias: None
Product: Virtualization Tools
Classification: Community
Component: libosinfo
Version: unspecified
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Fabiano Fidêncio
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2018-10-09 17:39 UTC by Cole Robinson
Modified: 2019-03-29 00:01 UTC (History)
4 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2019-03-29 00:01:13 UTC
Embargoed:


Attachments (Terms of Use)

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


Note You need to log in before you can comment on or make changes to this bug.