[Spice-devel] [PATCH] client: fix - exit on bad display-effect argument

Alon Levy alevy at redhat.com
Sun Aug 8 03:25:09 PDT 2010


----- "Yonit Halperin" <yhalperi at redhat.com> wrote:

> ---
>  client/application.cpp |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
> 
> diff --git a/client/application.cpp b/client/application.cpp
> index 3dafbc6..490cd8c 100644
> --- a/client/application.cpp
> +++ b/client/application.cpp
> @@ -1926,6 +1926,8 @@ bool
> Application::set_disabled_display_effects(CmdLineParser& parser, char
> *val,
>              disp_setting._disable_animation = true;
>          } else {
>              Platform::term_printf("%s: bad display effect type
> \"%s\"\n", arg0, val);
> +            _exit_code = SPICEC_ERROR_CODE_INVALID_ARG;
> +            return false;
>          }
>      } while ((val = parser.next_argument()));
>  
> -- 
> 1.7.1.1
> 

ACK

> _______________________________________________
> Spice-devel mailing list
> Spice-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/spice-devel


More information about the Spice-devel mailing list