[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#717621: lintian: apache2 AllowOverride matches obsolete Allow directive check



Control: tags -1 +patch

Attached is a patch:

It will check there is a white space type character, or a new line, after.

It will also check the directive is a the begining of a line, and will now
ignore comments and the name of a directive in a text, a url, and so on.

Note that all this space is out of $1, so that output is clean.
--- lintian-2.5.15/checks/apache2.pm	2013-07-05 19:41:14.000000000 +0000
+++ /usr/share/lintian/checks/apache2.pm	2013-07-23 22:12:41.735363437 +0000
@@ -230,7 +230,7 @@
     while (<$fd>)  {
 
         for my $directive ('Order', 'Satisfy', 'Allow', 'Deny', '<(|/)Limit.*?>', '<(|/)LimitExcept.*?>') {
-            if (m/($directive)/) {
+            if (m/^\s*($directive)(\s+|$)/) {
                 tag 'apache2-deprecated-auth-config', $1;
             }
         }

Reply to: