rc.d/jail: avoid misinterpreting expr arguments
ClosedPublic
Actions

Authored by emaste on Feb 27 2018, 5:55 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Nov 21, 7:48 AM

	Unknown Object (File)
	Fri, Nov 21, 7:47 AM

	Unknown Object (File)
	Fri, Nov 21, 7:45 AM

	Unknown Object (File)
	Fri, Nov 21, 7:43 AM

	Unknown Object (File)
	Thu, Nov 13, 11:00 PM

	Unknown Object (File)
	Tue, Nov 11, 2:17 AM

	Unknown Object (File)
	Sun, Oct 26, 12:16 AM

	Unknown Object (File)
	Sun, Oct 26, 12:16 AM

View All 73 Files

Subscribers

imp

Details

Reviewers

jamie
dteske

Commits

rS330122: rc.d/jail: avoid misinterpreting expr arguments

Summary

(Due to some misconfiguration) I ended up with _mask set to -v<something>, and /etc/rc.d/jail then failed with expr: illegal option -- v.

Use expr -- so that variable content is never interpreted as an option.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

emaste created this revision.Feb 27 2018, 5:55 PM

emaste edited the summary of this revision. (Show Details)

jamie accepted this revision.Feb 28 2018, 5:06 PM

This revision is now accepted and ready to land.Feb 28 2018, 5:06 PM

Closed by commit rS330122: rc.d/jail: avoid misinterpreting expr arguments (authored by emaste). · Explain WhyFeb 28 2018, 5:20 PM

This revision was automatically updated to reflect the committed changes.

Herald added a subscriber: imp. · View Herald TranscriptFeb 28 2018, 5:20 PM

Revision Contents
Changeset List

Path

Size

head/

etc/

rc.d/

jail

6 lines

Diff 39833

View Options

rc.d/jail: avoid misinterpreting expr argumentsClosedPublicActions