diff options
author | Jim Meyering <jim@meyering.net> | 2007-05-22 14:25:19 +0200 |
---|---|---|
committer | Jim Meyering <jim@meyering.net> | 2007-05-22 18:47:17 +0200 |
commit | a6a447fc58a01598682f0914f978d0a3c1cfc4dc (patch) | |
tree | 381e6cc2fecc1d4244daf27a8d2a67eccaf7c0df /src | |
parent | ae5717158f1b9f31b986b0f4416582684039ec55 (diff) | |
download | coreutils-a6a447fc58a01598682f0914f978d0a3c1cfc4dc.tar.xz |
cut: diagnose a range starting with 0 (-f 0-2) as invalid, and
give a better diagnostic for a field-number/offset of 0.
* NEWS: Mention the fix.
* src/cut.c (ADD_RANGE_PAIR): Add an explicit check for 0.
Based on a patch from James Youngman.
* tests/misc/cut: Add tests for the above.
Diffstat (limited to 'src')
-rw-r--r-- | src/cut.c | 2 |
1 files changed, 2 insertions, 0 deletions
@@ -57,6 +57,8 @@ #define ADD_RANGE_PAIR(rp, low, high) \ do \ { \ + if (low == 0 || high == 0) \ + FATAL_ERROR (_("fields and positions are numbered from 1")); \ if (n_rp >= n_rp_allocated) \ { \ (rp) = X2NREALLOC (rp, &n_rp_allocated); \ |