[virt-tools-list] [PATCH virt-viewer 1/6] win32: there is no CONERR$, only CONOUT$
Christophe Fergeau
cfergeau at redhat.com
Mon Oct 22 07:55:46 UTC 2012
Looks good, ACK.
Wouldn't this error have been caught earlier if there was error checking
in this code?
Christophe
On Sat, Oct 20, 2012 at 12:57:26AM +0200, Marc-André Lureau wrote:
> ---
> src/remote-viewer-main.c | 2 +-
> src/windows-cmdline-wrapper.c | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/remote-viewer-main.c b/src/remote-viewer-main.c
> index f92ef13..2a8973d 100644
> --- a/src/remote-viewer-main.c
> +++ b/src/remote-viewer-main.c
> @@ -233,7 +233,7 @@ main(int argc, char **argv)
> if (AttachConsole(ATTACH_PARENT_PROCESS) != 0) {
> freopen("CONIN$", "r", stdin);
> freopen("CONOUT$", "w", stdout);
> - freopen("CONERR$", "w", stderr);
> + freopen("CONOUT$", "w", stderr);
> dup2(fileno(stdin), STDIN_FILENO);
> dup2(fileno(stdout), STDOUT_FILENO);
> dup2(fileno(stderr), STDERR_FILENO);
> diff --git a/src/windows-cmdline-wrapper.c b/src/windows-cmdline-wrapper.c
> index 510763b..704e2d7 100644
> --- a/src/windows-cmdline-wrapper.c
> +++ b/src/windows-cmdline-wrapper.c
> @@ -30,7 +30,7 @@
> * if (AttachConsole(ATTACH_PARENT_PROCESS) != 0) {
> * freopen("CONIN$", "r", stdin);
> * freopen("CONOUT$", "w", stdout);
> - * freopen("CONERR$", "w", stderr);
> + * freopen("CONOUT$", "w", stderr);
> * dup2(fileno(stdin), STDIN_FILENO);
> * dup2(fileno(stdout), STDOUT_FILENO);
> * dup2(fileno(stderr), STDERR_FILENO);
> --
> 1.7.11.7
>
> _______________________________________________
> virt-tools-list mailing list
> virt-tools-list at redhat.com
> https://www.redhat.com/mailman/listinfo/virt-tools-list
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/virt-tools-list/attachments/20121022/2db335fc/attachment.sig>
More information about the virt-tools-list
mailing list