diff options
| author | Guillem Jover <guillem@debian.org> | 2012-03-31 20:30:54 +0200 |
|---|---|---|
| committer | Guillem Jover <guillem@debian.org> | 2012-03-31 21:43:30 +0200 |
| commit | 1a7d184fc65b91fc155d2724aad73b59aea8b9c8 (patch) | |
| tree | 1dd386484c6bd7c2adc2dc936593e5f84632de8c /src/t | |
| parent | 9305742ea75ee9b1fd58abf68c081b72692d3632 (diff) | |
| download | dpkg-1a7d184fc65b91fc155d2724aad73b59aea8b9c8.tar.gz | |
dpkg-divert: Check all dpkg-divert filename arguments
This makes sure the filename is absolute and does not contain newlines.
Closes: #21722
Diffstat (limited to 'src/t')
| -rw-r--r-- | src/t/100_dpkg_divert.t | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/src/t/100_dpkg_divert.t b/src/t/100_dpkg_divert.t index 93f97ad86..6d62fe561 100644 --- a/src/t/100_dpkg_divert.t +++ b/src/t/100_dpkg_divert.t @@ -33,7 +33,7 @@ if (! -x "@dd") { exit(0); } -plan tests => 239; +plan tests => 251; sub cleanup { system("rm -rf $tmpdir && mkdir -p $testdir"); @@ -151,8 +151,14 @@ call_divert_badusage(['--add', "/foo\nbar"], qr/newline/); call_divert_badusage(['--add', "$testdir"], qr/director(y|ies)/); call_divert_badusage(['--add', "--divert", "bar", "/foo/bar"], qr/absolute/); call_divert_badusage(['--remove'], qr/needs a single argument/); +call_divert_badusage(['--remove', 'foo'], qr/absolute/); +call_divert_badusage(['--remove', "/foo\nbar"], qr/newline/); call_divert_badusage(['--listpackage'], qr/needs a single argument/); +call_divert_badusage(['--listpackage', 'foo'], qr/absolute/); +call_divert_badusage(['--listpackage', "/foo\nbar"], qr/newline/); call_divert_badusage(['--truename'], qr/needs a single argument/); +call_divert_badusage(['--truename', 'foo'], qr/absolute/); +call_divert_badusage(['--truename', "/foo\nbar"], qr/newline/); call([@dd, '--admindir'], [], expect_failure => 1, expect_stderr_like => qr/(takes a value|needs.*argument)/); |