Package: check-all-the-things
Version: 2015.12.10
Severity: important
Tags: help security

The group selection enables checks that are to be disabled by default
due to having (dangerous/todo/etc flags). Checks with these flags
should only get enabled when explicitly requested. This is an important
issue because it enables dangerous checks when they weren't requested.
This problem still occurs in git master where groups/flags have been
merged. In git master it also enables checks that modify files. I have
tried to fix it but haven't been able to wrap my head around the
problem properly. I am hoping some of the folks subscribed to the bugs
for check-all-the-things can help fix this issue. If not I will try to
focus on fixing it during DebCamp. The below output illustrates this:

pabs@chianamo ~ $ mkdir tmp-test-cats-group-selection
pabs@chianamo ~ $ cd tmp-test-cats-group-selection/
pabs@chianamo ~/tmp-test-cats-group-selection $ cat 
/usr/share/check-all-the-things/data/perl
[perl-syntax-check]
apt = perl
match = *.pl *.pm
command = perl -wc {file} | grep -v ' syntax OK$'
comment = Dangerous because it executes code in use statements and BEGIN, 
UNITCHECK and CHECK blocks
flags = dangerous

[perl-b-lint]
apt = perl, libb-lint-perl
match = *.pl *.pm
prereq = perl -MO=Lint /dev/null
command = perl -MO=Lint {file} | grep -v ' syntax OK$'
comment = Dangerous because it executes code in use statements and BEGIN, 
UNITCHECK and CHECK blocks
flags = dangerous
...

pabs@chianamo ~/tmp-test-cats-group-selection $ /usr/bin/check-all-the-things 
-g perl
# Dangerous because it executes code in use statements and BEGIN, UNITCHECK and 
CHECK blocks
$ find -type f \( -iname '*.pl' -o -iname '*.pm' \) -exec perl -MO=Lint {} \; | 
grep -v ' syntax OK$'
./foo.pm syntax OK
./foo.pl syntax OK

# Dangerous because it executes code in use statements and BEGIN, UNITCHECK and 
CHECK blocks
$ find -type f \( -iname '*.pl' -o -iname '*.pm' \) -exec perl -wc {} \; | grep 
-v ' syntax OK$'
./foo.pm syntax OK
./foo.pl syntax OK
...

-- System Information:
Debian Release: stretch/sid
  APT prefers testing-debug
  APT policy: (900, 'testing-debug'), (900, 'testing'), (860, 
'testing-proposed-updates'), (800, 'unstable-debug'), (800, 'unstable'), (790, 
'buildd-unstable'), (700, 'experimental-debug'), (700, 'experimental'), (690, 
'buildd-experimental')
Architecture: amd64 (x86_64)

Kernel: Linux 4.6.0-trunk-amd64 (SMP w/4 CPU cores)
Locale: LANG=en_AU.utf8, LC_CTYPE=en_AU.utf8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)

Versions of packages check-all-the-things depends on:
ii  python3  3.5.1-3

Versions of packages check-all-the-things recommends:
ii  acheck                         0.5.2
ii  appstream                      0.9.6-1
pn  appstream-glib                 <none>
ii  bfbtester                      2.0.1-7.1
ii  blhc                           0.05-0.1
ii  bzip2                          1.0.6-8
ii  cabal-install                  1.22.7.0-1
ii  cbmc                           5.4-2
ii  clang                          1:3.6-33
ii  clang-modernize                1:3.6-33
ii  clang-tidy                     1:3.6-33
ii  cme                            1.011-1
ii  codespell                      1.8-1
ii  cppcheck                       1.73-1
ii  deheader                       1.3-1
ii  desktop-file-utils             0.22-1
ii  devscripts                     2.16.4
ii  dh-ocaml                       1.0.10
ii  duck                           0.9
ii  epubcheck                      4.0.1-2
ii  fdupes                         1.51-1
ii  flawfinder                     1.31-1
pn  fontforge-nox                  <none>
ii  freetype2-demos                2.6.3-3+b1
ii  gendarme                       4.2-1
ii  gettext                        0.19.7-2
ii  gettext-lint                   0.4-2.1
ii  ghc-mod                        5.4.0.0-1+b1
ii  golang-go                      2:1.6.1-2
ii  hlint                          1.9.26-1
ii  hopenpgp-tools                 0.18-1
ii  i18nspector                    0.24-1
ii  iwyu                           3.7-1
ii  jlint                          3.0-4.5+b1
ii  jpeginfo                       1.6.0-6+b2
ii  lacheck                        1.26-15
ii  libb-lint-perl                 1.20-1
ii  libconfig-model-dpkg-perl      2.079
ii  libconfig-model-perl           2.083-1
ii  libperl-critic-perl            1.126-1
ii  libpod-pom-perl                2.01-1
ii  libxml2-utils                  2.9.3+dfsg1-1
ii  lintex                         1.14-1
ii  lintian                        2.5.44
ii  lua-check                      0.15.0-1
ii  lzip                           1.17-1+b1
ii  lzop                           1.03-3.2
ii  moreutils                      0.58-1
ii  mp3check                       0.8.7-2+b1
ii  mp3val                         0.1.8-3+b1
ii  ocaml-nox                      4.02.3-6
ii  oggz-tools                     1.1.1-5
ii  opus-tools                     0.1.9-1
ii  p7zip                          15.14.1+dfsg-2
ii  pep8                           1.7.0-2
ii  perl                           5.22.2-1
ii  php5-cli                       5.6.22+dfsg-1
ii  pmccabe                        2.6
ii  pngcheck                       2.3.0-7
ii  puppet                         3.8.5-2
ii  puppet-lint                    1.1.0-1
ii  pyflakes                       1.2.3-1
ii  python-bashate                 0.3.1-2
ii  python-fontforge               20120731.b-7.2
ii  python-jpylyzer                1.17.0-1
ii  python-magic                   1:5.25-2
ii  python3-bashate                0.3.1-2
ii  python3-doc8                   0.6.0-3
ii  python3-magic                  1:5.25-2
ii  python3-ptyprocess             0.5.1-1
ii  python3-restructuredtext-lint  0.12.2-2
ii  sharutils                      1:4.15.2-1
ii  shellcheck                     0.3.7-5
ii  unzip                          6.0-20
ii  vorbis-tools                   1.4.0-10
ii  xz-utils                       5.1.1alpha+20120614-2.1

check-all-the-things suggests no packages.

-- no debconf information

-- 
bye,
pabs

https://wiki.debian.org/PaulWise

Attachment: signature.asc
Description: This is a digitally signed message part

Reply via email to