[PATCH weston v2] xdg-shell: Add invalid_parent error to xdg_popup

[Pekka Paalanen]

On Fri, 27 Feb 2015 18:37:41 +0800
Jonas Ådahl <jadahl at gmail.com> wrote:

> Send an invalid_parent error when the client tries to create a popup
> with a paren that is neither a xdg_surface nor a xdg_popup.
> 
> Signed-off-by: Jonas Ådahl <jadahl at gmail.com>
> ---
> 
> Changes since v1:
> 
> Removed unrelated diff.

Reviewed-by: Pekka Paalanen <pekka.paalanen at collabora.co.uk>


> 
>  desktop-shell/shell.c  | 11 ++++++++++-
>  protocol/xdg-shell.xml |  1 +
>  2 files changed, 11 insertions(+), 1 deletion(-)
> 
> diff --git a/desktop-shell/shell.c b/desktop-shell/shell.c
> index b326d68..bec49e6 100644
> --- a/desktop-shell/shell.c
> +++ b/desktop-shell/shell.c
> @@ -4092,12 +4092,21 @@ create_xdg_popup(struct shell_client *owner, void *shell,
>  		 uint32_t serial,
>  		 int32_t x, int32_t y)
>  {
> -	struct shell_surface *shsurf;
> +	struct shell_surface *shsurf, *parent_shsurf;
>  
>  	/* Verify that we are creating the top most popup when mapping,
>  	 * as its not until then we know whether it was mapped as most
>  	 * top level or not. */
>  
> +	parent_shsurf = get_shell_surface(parent);
> +	if (!shell_surface_is_xdg_popup(parent_shsurf) &&
> +	    !shell_surface_is_xdg_surface(parent_shsurf)) {
> +		wl_resource_post_error(owner->resource,
> +				       XDG_POPUP_ERROR_INVALID_PARENT,
> +				       "xdg_popup parent was invalid");
> +		return NULL;
> +	}
> +
>  	shsurf = create_common_surface(owner, shell, surface, client);
>  	if (!shsurf)
>  		return NULL;
> diff --git a/protocol/xdg-shell.xml b/protocol/xdg-shell.xml
> index 82ff917..fbf3723 100644
> --- a/protocol/xdg-shell.xml
> +++ b/protocol/xdg-shell.xml
> @@ -403,6 +403,7 @@
>  	These errors can be emitted in response to xdg_popup requests.
>        </description>
>        <entry name="not_the_topmost_popup" value="0" summary="The client tried to map or destroy a non-topmost popup"/>
> +      <entry name="invalid_parent" value="1" summary="The client specified an invalid parent surface"/>
>      </enum>
>  
>      <request name="destroy" type="destructor">