From 0321b694c7854579f4d21498c53d0f6381200703 Mon Sep 17 00:00:00 2001 From: Hiroki Sato Date: Wed, 26 Oct 2011 02:11:28 +0000 Subject: [PATCH] - Add support for a "!" character in regex matching in devd(8). It inverts the logic (true/false) of the matching. - Add "!usbus[0-9]+" to IFNET ATTACH notification handler in the default devd.conf to prevent rc.d/netif from running when usbus[0-9]+ is attached. Reviewed by: imp --- etc/devd.conf | 1 + sbin/devd/devd.cc | 16 +++++++++++++--- sbin/devd/devd.conf.5 | 6 +++++- sbin/devd/devd.hh | 1 + 4 files changed, 20 insertions(+), 4 deletions(-) diff --git a/etc/devd.conf b/etc/devd.conf index 2ba2e0efadc7..27abc1f597a7 100644 --- a/etc/devd.conf +++ b/etc/devd.conf @@ -38,6 +38,7 @@ options { # notify 0 { match "system" "IFNET"; + match "subsystem" "!usbus[0-9]+"; match "type" "ATTACH"; action "/etc/pccard_ether $subsystem start"; }; diff --git a/sbin/devd/devd.cc b/sbin/devd/devd.cc index 87d0580e26d4..fd3b6e396785 100644 --- a/sbin/devd/devd.cc +++ b/sbin/devd/devd.cc @@ -251,7 +251,14 @@ match::match(config &c, const char *var, const char *re) : _var(var) { _re = "^"; - _re.append(c.expand_string(string(re))); + if (!c.expand_string(string(re)).empty() && + c.expand_string(string(re)).at(0) == '!') { + _re.append(c.expand_string(string(re)).substr(1)); + _inv = 1; + } else { + _re.append(c.expand_string(string(re))); + _inv = 0; + } _re.append("$"); regcomp(&_regex, _re.c_str(), REG_EXTENDED | REG_NOSUB | REG_ICASE); } @@ -268,10 +275,13 @@ match::do_match(config &c) bool retval; if (Dflag) - fprintf(stderr, "Testing %s=%s against %s\n", _var.c_str(), - value.c_str(), _re.c_str()); + fprintf(stderr, "Testing %s=%s against %s, invert=%d\n", + _var.c_str(), value.c_str(), _re.c_str(), _inv); retval = (regexec(&_regex, value.c_str(), 0, NULL, 0) == 0); + if (_inv == 1) + retval = (retval == 0) ? 1 : 0; + return retval; } diff --git a/sbin/devd/devd.conf.5 b/sbin/devd/devd.conf.5 index e4dea72fd317..09e902bf188e 100644 --- a/sbin/devd/devd.conf.5 +++ b/sbin/devd/devd.conf.5 @@ -41,7 +41,7 @@ .\" ACTION, ARISING OUT OF OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS .\" SOFTWARE. .\" -.Dd March 8, 2009 +.Dd October 25, 2011 .Dt DEVD.CONF 5 .Os .Sh NAME @@ -121,6 +121,10 @@ Creates a regular expression and assigns it to the variable .Ar regexp-name . The variable is available throughout the rest of the configuration file. +If the string begins with +.Ql \&! , +it matches if the regular expression formed by the rest of the string +does not match. All regular expressions have an implicit .Ql ^$ around them. diff --git a/sbin/devd/devd.hh b/sbin/devd/devd.hh index b3c98f3ebe0c..d70d1787e452 100644 --- a/sbin/devd/devd.hh +++ b/sbin/devd/devd.hh @@ -92,6 +92,7 @@ public: private: std::string _var; std::string _re; + bool _inv; regex_t _regex; };