Bug 1097453

Summary:	Greedy regex in rhel-readonly matches non-path characters
Product:	Red Hat Enterprise Linux 7	Reporter:	Ryan Barry <rbarry>
Component:	initscripts	Assignee:	initscripts Maintenance Team <initscripts-maint-list>
Status:	CLOSED ERRATA	QA Contact:	qe-baseos-daemons
Severity:	unspecified	Docs Contact:
Priority:	urgent
Version:	7.0	CC:	bmcclain, dfediuck, fdeutsch, jkurik, jscotka, leiwang, lnykryn, lpol, ycui
Target Milestone:	rc	Keywords:	ZStream
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	initscripts-9.49.18-1.el7	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:
Clones:	1130112 (view as bug list)		Environment:
Last Closed:	2015-03-05 10:52:33 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:
Bug Depends On:
Bug Blocks:	1095140, 1130112

Description Ryan Barry 2014-05-13 22:04:27 UTC

Description of problem:

Greedy regex in rhel-readonly prevents proper operation of rwtab.

Version-Release number of selected component (if applicable):

Any

How reproducible:

100%

Steps to Reproduce:
1. Add /boot-anything to rwtab (or anything else which starts with a mountpoint)


Actual results:
Directory is mounted readonly

Expected results:
Directory is mounted readwrite

Additional info:
This appears to be a greedy regex at line 112 of rhel-readonly which is matching characters which are not path separators.

Comment 2 Lukáš Nykrýn 2014-05-14 10:21:47 UTC

Should be already fixed in upstream.

https://git.fedorahosted.org/cgit/initscripts.git/commit/?id=32e8cbc1c91ada764ef4b5bb817b0c045d3fb3a4

Comment 3 Ryan Barry 2014-05-14 14:09:10 UTC

Great! When can we expect it to land in 7?

Comment 4 Lukáš Nykrýn 2014-05-15 07:09:40 UTC

7.1 unless there will be a request for z-stream

Comment 5 Fabian Deutsch 2014-05-15 09:20:30 UTC

Okay, here is our request for rhel-7.0.z

It is crucial for us to get this early to allow decent testing of RHEV-H.

Comment 7 Lukáš Nykrýn 2014-07-24 11:57:03 UTC

I am really sorry, I have completely overlooked that this is proposed to z-stream.

https://git.fedorahosted.org/cgit/initscripts.git/commit/?id=bdfa5a5

Comment 12 errata-xmlrpc 2015-03-05 10:52:33 UTC

Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://rhn.redhat.com/errata/RHBA-2015-0471.html