[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 3/6] json-parser: Fix two coverity defects
From: |
Luiz Capitulino |
Subject: |
Re: [Qemu-devel] [PATCH v2 3/6] json-parser: Fix two coverity defects |
Date: |
Mon, 9 Jun 2014 16:15:48 -0400 |
On Wed, 28 May 2014 21:21:37 +0800
<address@hidden> wrote:
> From: Gonglei <address@hidden>
>
> Pointer "token" returned by "parser_context_pop_token(ctxt)" is never used.
That's not true, "token" is used by token_is_operator(). The patch seems
correct to me, but you have to fix the changelog.
Also, who's going to take care of this series? I'm willing to cherry-pick
this patch...
>
> Signed-off-by: ChenLiang <address@hidden>
> Signed-off-by: Gonglei <address@hidden>
> ---
> qobject/json-parser.c | 15 ++-------------
> 1 file changed, 2 insertions(+), 13 deletions(-)
>
> diff --git a/qobject/json-parser.c b/qobject/json-parser.c
> index e46c264..4288267 100644
> --- a/qobject/json-parser.c
> +++ b/qobject/json-parser.c
> @@ -423,7 +423,6 @@ static QObject *parse_object(JSONParserContext *ctxt,
> va_list *ap)
> if (!token_is_operator(token, '{')) {
> goto out;
> }
> - token = NULL;
>
> dict = qdict_new();
>
> @@ -449,7 +448,6 @@ static QObject *parse_object(JSONParserContext *ctxt,
> va_list *ap)
> parse_error(ctxt, token, "expected separator in dict");
> goto out;
> }
> - token = NULL;
>
> if (parse_pair(ctxt, dict, ap) == -1) {
> goto out;
> @@ -461,10 +459,8 @@ static QObject *parse_object(JSONParserContext *ctxt,
> va_list *ap)
> goto out;
> }
> }
> - token = NULL;
> } else {
> - token = parser_context_pop_token(ctxt);
> - token = NULL;
> + (void)parser_context_pop_token(ctxt);
> }
>
> return QOBJECT(dict);
> @@ -487,10 +483,8 @@ static QObject *parse_array(JSONParserContext *ctxt,
> va_list *ap)
> }
>
> if (!token_is_operator(token, '[')) {
> - token = NULL;
> goto out;
> }
> - token = NULL;
>
> list = qlist_new();
>
> @@ -523,8 +517,6 @@ static QObject *parse_array(JSONParserContext *ctxt,
> va_list *ap)
> goto out;
> }
>
> - token = NULL;
> -
> obj = parse_value(ctxt, ap);
> if (obj == NULL) {
> parse_error(ctxt, token, "expecting value");
> @@ -539,11 +531,8 @@ static QObject *parse_array(JSONParserContext *ctxt,
> va_list *ap)
> goto out;
> }
> }
> -
> - token = NULL;
> } else {
> - token = parser_context_pop_token(ctxt);
> - token = NULL;
> + (void)parser_context_pop_token(ctxt);
> }
>
> return QOBJECT(list);
- Re: [Qemu-devel] [PATCH v2 3/6] json-parser: Fix two coverity defects,
Luiz Capitulino <=