On Wed, Jul 10, 2013 at 11:40:56PM +0200, Mariusz Ceier wrote:
> Functions like wl_argument_from_va_list expect from get_next_argument,
> to initialize details->type but when the signature is empty or contains
> only version (like in desktop-shell-protocol.c in weston) it is left
> uninitialized.
> 
> This patch fixes it, by initializing details->type with '\0' value,
> signaling end of arguments.
> 
> Signed-off-by: Mariusz Ceier <[email protected]>

Good catch, thanks.

Kristian

> ---
>  src/connection.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/src/connection.c b/src/connection.c
> index 2ca9bce..9bb850c 100644
> --- a/src/connection.c
> +++ b/src/connection.c
> @@ -419,6 +419,7 @@ get_next_argument(const char *signature, struct 
> argument_details *details)
>                       details->nullable = 1;
>               }
>       }
> +     details->type = '\0';
>       return signature;
>  }
>  
> -- 
> 1.8.3.2
> 
> _______________________________________________
> wayland-devel mailing list
> [email protected]
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel
_______________________________________________
wayland-devel mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/wayland-devel

Reply via email to