summaryrefslogtreecommitdiff
path: root/tests/misc/shred-passes.sh
diff options
context:
space:
mode:
authorPádraig Brady <P@draigBrady.com>2013-11-07 12:32:29 +0000
committerPádraig Brady <P@draigBrady.com>2013-11-08 13:39:35 +0000
commit90181a5926e96f34b5ffff21b81a2874a846ff6f (patch)
tree6cdda1ab6f90c873542c11de9a6d1b9dab667242 /tests/misc/shred-passes.sh
parentc2eaef2f57153a22b6dc075b1adc9fde1a4eb7bd (diff)
downloadcoreutils-90181a5926e96f34b5ffff21b81a2874a846ff6f.tar.xz
shred: avoid data write pass with --size=0
* src/shred.c (dopass): Exit early to avoid redundant heap allocation, and more importantly avoiding a file sync when we're writting no data, as this can have side effects. Also with --verbose, this avoids printing of "pass status" which could be confusing as to whether data was actually written. * tests/misc/shred-passes.sh: Ensure the status for data passes are not written when not doing any data writes.
Diffstat (limited to 'tests/misc/shred-passes.sh')
-rwxr-xr-xtests/misc/shred-passes.sh14
1 files changed, 13 insertions, 1 deletions
diff --git a/tests/misc/shred-passes.sh b/tests/misc/shred-passes.sh
index cfdd68e98..938683bce 100755
--- a/tests/misc/shred-passes.sh
+++ b/tests/misc/shred-passes.sh
@@ -31,9 +31,21 @@ shred: f: removing
shred: f: renamed to 0
shred: f: removed" > exp || framework_failure_
-
shred -v -u f 2>out || fail=1
compare exp out || fail=1
+# Likewise but with --exact to bypass the
+# data passes for the zero length file
+touch f || framework_failure_
+echo "\
+shred: f: removing
+shred: f: renamed to 0
+shred: f: removed" > exp || framework_failure_
+
+shred -x -v -u f 2>out || fail=1
+
+compare exp out || fail=1
+
+
Exit $fail