[OpenWrt-Devel] [PATCH v2 08/10] json_script: fix logic invert of handle_expr_not().

Felix Fietkau nbd at openwrt.org
Thu Dec 11 11:52:39 EST 2014


On 2014-11-12 14:59, Yousong Zhou wrote:
> Signed-off-by: Yousong Zhou <yszhou4tech at gmail.com>
> ---
>  json_script.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/json_script.c b/json_script.c
> index 0ec7b38..0d51f79 100644
> --- a/json_script.c
> +++ b/json_script.c
> @@ -338,7 +338,7 @@ static int handle_expr_not(struct json_call *call, struct blob_attr *expr)
>  	if (!tb[1])
>  		return -1;
>  
> -	return json_process_expr(call, tb[1]);
> +	return !json_process_expr(call, tb[1]);
Simply adding ! is not enough. This needs to check for ret < 0 for
proper error handling.

- Felix
_______________________________________________
openwrt-devel mailing list
openwrt-devel at lists.openwrt.org
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel



More information about the openwrt-devel mailing list