Le 19/03/2015 09:58, Jonas Ådahl a écrit :
> Without this patch, the scanner would generate invalid C which wouldn't
> compile anyway, so lets be nice and fail earlier and point out where the
> error is.
> 
> Signed-off-by: Jonas Ådahl <[email protected]>

+1

Reviewed-By: David FORT <[email protected]>

> ---
>  src/scanner.c | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/src/scanner.c b/src/scanner.c
> index 1f1e59a..64431a1 100644
> --- a/src/scanner.c
> +++ b/src/scanner.c
> @@ -551,6 +551,10 @@ end_element(void *data, const XML_Char *name)
>                  strcmp(name, "event") == 0) {
>               ctx->message = NULL;
>       } else if (strcmp(name, "enum") == 0) {
> +             if (wl_list_empty(&ctx->enumeration->entry_list)) {
> +                     fail(&ctx->loc, "enumeration %s was empty",
> +                          ctx->enumeration->name);
> +             }
>               ctx->enumeration = NULL;
>       }
>  }
> 


-- 
David FORT
website: http://www.hardening-consulting.com/

_______________________________________________
wayland-devel mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/wayland-devel

Reply via email to