[BusyBox-cvs] CVS update of busybox/coreutils (seq.c)

Glenn McGrath bug1 at codepoet.org
Fri Jul 23 06:43:30 UTC 2004


    Date: Friday, July 23, 2004 @ 00:43:30
  Author: bug1
    Path: /var/cvs/busybox/coreutils

Modified: seq.c (1.2 -> 1.3)

Patch from Felipe Kellermann, fix endless loop when first > last and 
increment > 0.


Index: busybox/coreutils/seq.c
diff -u busybox/coreutils/seq.c:1.2 busybox/coreutils/seq.c:1.3
--- busybox/coreutils/seq.c:1.2	Wed Feb  4 04:01:19 2004
+++ busybox/coreutils/seq.c	Fri Jul 23 00:43:29 2004
@@ -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 @@
 	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;
 }



More information about the busybox-cvs mailing list