diff options
author | Glenn L McGrath <bug1@ihug.co.nz> | 2004-07-23 10:43:29 +0400 |
---|---|---|
committer | Glenn L McGrath <bug1@ihug.co.nz> | 2004-07-23 10:43:29 +0400 |
commit | efc6bf63651496ad5a87aa8b902615e4949c0e3e (patch) | |
tree | fa4f1776402e700ed915f9b2363099c42bda8e84 /coreutils | |
parent | c0dd26f68d3b7956beed6f32088f73356fe2aa16 (diff) |
Patch from Felipe Kellermann, fix endless loop when first > last and
increment > 0.
Diffstat (limited to 'coreutils')
-rw-r--r-- | coreutils/seq.c | 19 |
1 files changed, 13 insertions, 6 deletions
diff --git a/coreutils/seq.c b/coreutils/seq.c index 8a2a80c14..8006be83d 100644 --- a/coreutils/seq.c +++ b/coreutils/seq.c @@ -1,4 +1,7 @@ +/* vi: set sw=4 ts=4: */ /* + * seq implementation for busybox + * * This program is free software; you can redistribute it and/or modify * it under the terms of version 2 of the GNU General Public License as * published by the Free Software Foundation. @@ -27,18 +30,22 @@ extern int seq_main(int argc, char **argv) if (argc == 4) { first = atof(argv[1]); increment = atof(argv[2]); - } - else if (argc == 3) { + } else if (argc == 3) { first = atof(argv[1]); - } - else if (argc != 2) { + } else if (argc != 2) { bb_show_usage(); } last = atof(argv[argc - 1]); - for (i = first; ((first <= last) ? (i <= last): (i >= last));i += increment) { + /* You should note that this is pos-5.0.91 semantics, -- FK. */ + if ((first > last) && (increment > 0)) { + return EXIT_SUCCESS; + } + + for (i = first; ((first <= last) ? (i <= last) : (i >= last)); + i += increment) { printf("%g\n", i); } - return(EXIT_SUCCESS); + return EXIT_SUCCESS; } |