Package: devscripts
Version: 2.10.71
Severity: minor
User: devscri...@packages.debian.org
Usertags: debdiff

Hi,

Debdiff's handling of files that do not end on \.(u?deb|dsc|changes)$ is:
a) undocumented
b) bogus, as one may end up comparing a .changes and a .deb

"b)" can be triggered by running:
debdiff deb_file_that_doesn\'t_end_in_common_exts a.changes

`file $ARGV[0]` =~ /Debian/ (in line 321) sets $type to 'deb', but the check 
for the second file doesn't fail because there's an incorrect exclusion:
unless ($type eq 'deb' and `file $ARGV[0]` =~ /Debian/) {

I guess the second call to file(1) was meant to check $ARGV[1],

Cheers,
-- 
Raphael Geissert - Debian Developer
www.debian.org - get.debian.net



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to