Pavel fixed a segfault in yes.

 -Erik

3 files changed, 3 insertions, 2 deletions

diff --git a/Changelog b/Changelog
index 3083ae960..829232911 100644
--- a/Changelog
+++ b/Changelog
@@ -48,6 +48,7 @@
         * Fixed more and ps to have sensible terminal width defaults, thanks 
             to Pavel Roskin.
         * Fixed all fatalError() calls lacking a "\n", thanks to Pavel Roskin.
+        * Fixed a segfault in yes when no args were given -- Pavel Roskin.
         * More doc updates
 
 
diff --git a/coreutils/yes.c b/coreutils/yes.c
index 97b6f653c..1718af4bb 100644
--- a/coreutils/yes.c
+++ b/coreutils/yes.c
@@ -27,7 +27,7 @@ extern int yes_main(int argc, char **argv)
 {
         int i;
 
-        if (argc >=1 && *argv[1]=='-') {
+        if (argc >= 2 && *argv[1] == '-') {
                 usage("yes [OPTION]... [STRING]...\n"
 #ifndef BB_FEATURE_TRIVIAL_HELP
                                 "\nRepeatedly outputs a line with all specified STRING(s), or `y'.\n"
diff --git a/yes.c b/yes.c
index 97b6f653c..1718af4bb 100644
--- a/yes.c
+++ b/yes.c
@@ -27,7 +27,7 @@ extern int yes_main(int argc, char **argv)
 {
         int i;
 
-        if (argc >=1 && *argv[1]=='-') {
+        if (argc >= 2 && *argv[1] == '-') {
                 usage("yes [OPTION]... [STRING]...\n"
 #ifndef BB_FEATURE_TRIVIAL_HELP
                                 "\nRepeatedly outputs a line with all specified STRING(s), or `y'.\n"