Re: [PATCH] Before calling execvp(), reset needs to flush stdout.

[Denys Vlasenko]

Applied, thanks

On Thu, Feb 16, 2017 at 3:45 PM, Glenn Matthews (glmatthe)
 wrote:
> Before calling execvp(), reset needs to flush stdout.
>
> Otherwise the reset sequence gets dropped on the floor.
> ---
>  AUTHORS   | 3 +++
>  console-tools/reset.c | 2 ++
>  2 files changed, 5 insertions(+)
>
> diff --git a/AUTHORS b/AUTHORS
> index fa58697..9410c5e 100644
> --- a/AUTHORS
> +++ b/AUTHORS
> @@ -95,6 +95,9 @@ Stephan Linz 
>  John Lombardo 
>  tr
>
> +Glenn Matthews 
> +Bug fix(es).
> +
>  Glenn McGrath 
>  Common unarchiving code and unarchiving applets, ifupdown, ftpgetput,
>  nameif, sed, patch, fold, install, uudecode.
> diff --git a/console-tools/reset.c b/console-tools/reset.c
> index 57cebb4..7c7b99d 100644
> --- a/console-tools/reset.c
> +++ b/console-tools/reset.c
> @@ -56,6 +56,8 @@ int reset_main(int argc UNUSED_PARAM, char **argv 
> UNUSED_PARAM)
>  #if ENABLE_STTY
> return stty_main(2, (char**)args);
>  #else
> +   /* Make sure stdout gets drained before we execvp */
> +   fflush(stdout);
> execvp("stty", (char**)args);
>  #endif
> }
> --
> 2.9.3
>
>
>
>
> ___
> busybox mailing list
> busybox@busybox.net
> http://lists.busybox.net/mailman/listinfo/busybox
___
busybox mailing list
busybox@busybox.net
http://lists.busybox.net/mailman/listinfo/busybox

[PATCH] Before calling execvp(), reset needs to flush stdout.

[Glenn Matthews (glmatthe)]

Before calling execvp(), reset needs to flush stdout.

Otherwise the reset sequence gets dropped on the floor.
---
 AUTHORS   | 3 +++
 console-tools/reset.c | 2 ++
 2 files changed, 5 insertions(+)

diff --git a/AUTHORS b/AUTHORS
index fa58697..9410c5e 100644
--- a/AUTHORS
+++ b/AUTHORS
@@ -95,6 +95,9 @@ Stephan Linz 
 John Lombardo 
 tr
 
+Glenn Matthews 
+Bug fix(es).
+
 Glenn McGrath 
 Common unarchiving code and unarchiving applets, ifupdown, ftpgetput,
 nameif, sed, patch, fold, install, uudecode.
diff --git a/console-tools/reset.c b/console-tools/reset.c
index 57cebb4..7c7b99d 100644
--- a/console-tools/reset.c
+++ b/console-tools/reset.c
@@ -56,6 +56,8 @@ int reset_main(int argc UNUSED_PARAM, char **argv 
UNUSED_PARAM)
 #if ENABLE_STTY
return stty_main(2, (char**)args);
 #else
+   /* Make sure stdout gets drained before we execvp */
+   fflush(stdout);
execvp("stty", (char**)args);
 #endif
}
-- 
2.9.3




___
busybox mailing list
busybox@busybox.net
http://lists.busybox.net/mailman/listinfo/busybox