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

Bug#911157: lintian: complain about grepping the passwd/group file instead of using getent



Hi Rhonda,

> ~\b(grep\b.*/etc/(?:passwd|group))\b
> 
>  I'm not completely sure about the syntax here, but the \b before the
> bracket looks like it wouldn't catch egrep

Oh, good catch. Fixed in:

  https://salsa.debian.org/lintian/lintian/commit/38b74fe0a101aa10d4ea87084fa05bc420986321

I decided against using just the filename itself as there were a few
too many false-positives and I think we would catch most peccant
maintainers of new packages it this way.

See, for example:

  https://codesearch.debian.net/search?q=%2Fetc%2Fpasswd+path%3Adebian%2F*post*

and

  https://codesearch.debian.net/search?q=%2Fetc%2Fpasswd+path%3Adebian%2F*pre*


Regards,

-- 
      ,''`.
     : :'  :     Chris Lamb
     `. `'`      lamby@debian.org / chris-lamb.co.uk
       `-


Reply to: