From a9d63d29a1ce7ed4d1d5575308a0c72568381c99 Mon Sep 17 00:00:00 2001 From: Andrew Dolgov Date: Thu, 11 Nov 2010 14:06:08 +0300 Subject: [PATCH] get_article_filters: silence preg_match() warnings on incorrect user input --- functions.php | 24 ++++++++++++------------ 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/functions.php b/functions.php index 0ae97af4..126ad27c 100644 --- a/functions.php +++ b/functions.php @@ -1520,8 +1520,8 @@ foreach ($filters["title"] as $filter) { $reg_exp = $filter["reg_exp"]; $inverse = $filter["inverse"]; - if ((!$inverse && preg_match("/$reg_exp/i", $title)) || - ($inverse && !preg_match("/$reg_exp/i", $title))) { + if ((!$inverse && @preg_match("/$reg_exp/i", $title)) || + ($inverse && !@preg_match("/$reg_exp/i", $title))) { array_push($matches, array($filter["action"], $filter["action_param"])); } @@ -1533,8 +1533,8 @@ $reg_exp = $filter["reg_exp"]; $inverse = $filter["inverse"]; - if ((!$inverse && preg_match("/$reg_exp/i", $content)) || - ($inverse && !preg_match("/$reg_exp/i", $content))) { + if ((!$inverse && @preg_match("/$reg_exp/i", $content)) || + ($inverse && !@preg_match("/$reg_exp/i", $content))) { array_push($matches, array($filter["action"], $filter["action_param"])); } @@ -1547,11 +1547,11 @@ $inverse = $filter["inverse"]; if ($inverse) { - if (!preg_match("/$reg_exp/i", $title) && !preg_match("/$reg_exp/i", $content)) { + if (!@preg_match("/$reg_exp/i", $title) && !preg_match("/$reg_exp/i", $content)) { array_push($matches, array($filter["action"], $filter["action_param"])); } } else { - if (preg_match("/$reg_exp/i", $title) || preg_match("/$reg_exp/i", $content)) { + if (@preg_match("/$reg_exp/i", $title) || preg_match("/$reg_exp/i", $content)) { array_push($matches, array($filter["action"], $filter["action_param"])); } } @@ -1564,8 +1564,8 @@ $reg_exp = $filter["reg_exp"]; $inverse = $filter["inverse"]; - if ((!$inverse && preg_match("/$reg_exp/i", $link)) || - ($inverse && !preg_match("/$reg_exp/i", $link))) { + if ((!$inverse && @preg_match("/$reg_exp/i", $link)) || + ($inverse && !@preg_match("/$reg_exp/i", $link))) { array_push($matches, array($filter["action"], $filter["action_param"])); } @@ -1602,8 +1602,8 @@ foreach ($filters["author"] as $filter) { $reg_exp = $filter["reg_exp"]; $inverse = $filter["inverse"]; - if ((!$inverse && preg_match("/$reg_exp/i", $author)) || - ($inverse && !preg_match("/$reg_exp/i", $author))) { + if ((!$inverse && @preg_match("/$reg_exp/i", $author)) || + ($inverse && !@preg_match("/$reg_exp/i", $author))) { array_push($matches, array($filter["action"], $filter["action_param"])); } @@ -1618,8 +1618,8 @@ $reg_exp = $filter["reg_exp"]; $inverse = $filter["inverse"]; - if ((!$inverse && preg_match("/$reg_exp/i", $tag_string)) || - ($inverse && !preg_match("/$reg_exp/i", $tag_string))) { + if ((!$inverse && @preg_match("/$reg_exp/i", $tag_string)) || + ($inverse && !@preg_match("/$reg_exp/i", $tag_string))) { array_push($matches, array($filter["action"], $filter["action_param"])); } -- 2.39.5