From bfffa909477924eaaa2a5e20ee44be087b0ac87d Mon Sep 17 00:00:00 2001
From: =?utf8?q?P=C3=A1draig=20Brady?= <P@draigBrady.com>
Date: Fri, 20 May 2011 01:18:28 +0100
Subject: [PATCH] split: diagnose when --filter is used with a chunk number

* src/split.c (main): Exit with a diagnostic if --filter
is specified along with a specific chunk number.
* test/split/filter: Ensure this combination fails.
---
 src/split.c        | 6 ++++++
 tests/split/filter | 3 +++
 2 files changed, 9 insertions(+)

diff --git a/src/split.c b/src/split.c
index 5e203f1..9e23f86 100644
--- a/src/split.c
+++ b/src/split.c
@@ -1175,6 +1175,12 @@ main (int argc, char **argv)
         }
     }
 
+  if (k_units != 0 && filter_command)
+    {
+      error (0, 0, _("--filter does not process a chunk extracted to stdout"));
+      usage (EXIT_FAILURE);
+    }
+
   /* Handle default case.  */
   if (split_type == type_undef)
     {
diff --git a/tests/split/filter b/tests/split/filter
index a42c553..5bc5458 100755
--- a/tests/split/filter
+++ b/tests/split/filter
@@ -43,6 +43,9 @@ done
 split -e -n 10 --filter='xz > $FILE.xz' /dev/null || fail=1
 stat x?? 2>/dev/null && fail=1
 
+# Ensure this invalid combination is flagged
+split -n 1/2 --filter='true' /dev/null 2>/dev/null && fail=1
+
 # Ensure SIGPIPEs sent by the children don't propagate back
 # where they would result in a non zero exit from split.
 yes | head -n200K | split -b1G --filter='head -c1 >/dev/null' || fail=1
-- 
2.7.4