Quoting marcandre.lur...@redhat.com (2016-02-17 10:47:54)
> From: Marc-André Lureau <marcandre.lur...@redhat.com>
>
> Laszlo Ersek said: "The length check is off by one (in the safe direction); it
> should be (nchars >= 2). The processing should be active for the wide string
> L"\r\n" -- resulting in the empty wide string --, I believe."
>
> Reported-by: Laszlo Ersek <ler...@redhat.com>
> Signed-off-by: Marc-André Lureau <marcandre.lur...@redhat.com>
Reviewed-by: Michael Roth <mdr...@linux.vnet.ibm.com>
> ---
> qga/commands-win32.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/qga/commands-win32.c b/qga/commands-win32.c
> index b20f107..ae8cf3d 100644
> --- a/qga/commands-win32.c
> +++ b/qga/commands-win32.c
> @@ -1266,7 +1266,7 @@ get_net_error_message(gint error)
> if (msg != NULL) {
> nchars = wcslen(msg);
>
> - if (nchars > 2 &&
> + if (nchars >= 2 &&
> msg[nchars - 1] == L'\n' &&
> msg[nchars - 2] == L'\r') {
> msg[nchars - 2] = L'\0';
> --
> 2.5.0
>
