[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH V3 02/13] libxl_json: Check the parser status before
From: |
Anthony PERARD |
Subject: |
[Qemu-devel] [PATCH V3 02/13] libxl_json: Check the parser status before to call parse_complete |
Date: |
Tue, 1 Nov 2011 16:07:15 +0000 |
Also, use goto to handle an error.
Signed-off-by: Anthony PERARD <address@hidden>
---
tools/libxl/libxl_json.c | 37 ++++++++++++++++++++-----------------
1 files changed, 20 insertions(+), 17 deletions(-)
diff --git a/tools/libxl/libxl_json.c b/tools/libxl/libxl_json.c
index 11f65fc..389b697 100644
--- a/tools/libxl/libxl_json.c
+++ b/tools/libxl/libxl_json.c
@@ -730,6 +730,8 @@ libxl__json_object *libxl__json_parse(libxl__gc *gc, const
char *s)
{
yajl_status status;
libxl__yajl_ctx yajl_ctx;
+ libxl__json_object *o = NULL;
+ unsigned char *str = NULL;
memset(&yajl_ctx, 0, sizeof (yajl_ctx));
yajl_ctx.gc = gc;
@@ -744,30 +746,31 @@ libxl__json_object *libxl__json_parse(libxl__gc *gc,
const char *s)
yajl_ctx.hand = yajl_alloc(&callbacks, &cfg, NULL, &yajl_ctx);
}
status = yajl_parse(yajl_ctx.hand, (const unsigned char *)s, strlen(s));
+ if (status != yajl_status_ok)
+ goto out;
+
status = yajl_parse_complete(yajl_ctx.hand);
+ if (status != yajl_status_ok)
+ goto out;
- if (status == yajl_status_ok) {
- libxl__json_object *o = yajl_ctx.head;
+ o = yajl_ctx.head;
- DEBUG_GEN_REPORT(&yajl_ctx);
+ DEBUG_GEN_REPORT(&yajl_ctx);
- yajl_ctx.head = NULL;
+ yajl_ctx.head = NULL;
- yajl_ctx_free(&yajl_ctx);
- return o;
- } else {
- unsigned char *str = yajl_get_error(yajl_ctx.hand, 1,
- (const unsigned char *)s,
- strlen(s));
+ yajl_ctx_free(&yajl_ctx);
+ return o;
- LIBXL__LOG(libxl__gc_owner(gc), LIBXL__LOG_ERROR,
- "yajl error: %s", str);
- yajl_free_error(yajl_ctx.hand, str);
+out:
+ str = yajl_get_error(yajl_ctx.hand, 1, (const unsigned char*)s, strlen(s));
- libxl__json_object_free(gc, yajl_ctx.head);
- yajl_ctx_free(&yajl_ctx);
- return NULL;
- }
+ LIBXL__LOG(libxl__gc_owner(gc), LIBXL__LOG_ERROR, "yajl error: %s", str);
+ yajl_free_error(yajl_ctx.hand, str);
+
+ libxl__json_object_free(gc, yajl_ctx.head);
+ yajl_ctx_free(&yajl_ctx);
+ return NULL;
}
static const char *yajl_gen_status_to_string(yajl_gen_status s)
--
Anthony PERARD
- [Qemu-devel] [PATCH V3 00/13] libxl: QMP client improvement + pci passthrougth insert through QMP, Anthony PERARD, 2011/11/01
- [Qemu-devel] [PATCH V3 03/13] libxl_qmp: Better error message after a parse error., Anthony PERARD, 2011/11/01
- [Qemu-devel] [PATCH V3 05/13] libxl_qmp: Introduce an opaque argument to the callbacks., Anthony PERARD, 2011/11/01
- [Qemu-devel] [PATCH V3 01/13] libxl_qmp: Fix return check of fcntl, Anthony PERARD, 2011/11/01
- [Qemu-devel] [PATCH V3 08/13] libxl_qmp: Introduce qmp_request_context., Anthony PERARD, 2011/11/01
- [Qemu-devel] [PATCH V3 10/13] libxl_qmp: Introduce libxl__qmp_pci_add., Anthony PERARD, 2011/11/01
- [Qemu-devel] [PATCH V3 02/13] libxl_json: Check the parser status before to call parse_complete,
Anthony PERARD <=
- [Qemu-devel] [PATCH V3 11/13] libxl: Use QMP to insert a passthrough device when using upstream QEMU, Anthony PERARD, 2011/11/01
- [Qemu-devel] [PATCH V3 13/13] libxl: Remove a passthrough device through QMP., Anthony PERARD, 2011/11/01
- [Qemu-devel] [PATCH V3 06/13] libxl_qmp: Introduce list of arguments to qmp_send, Anthony PERARD, 2011/11/01
- [Qemu-devel] [PATCH V3 09/13] libxl_json: Handle number abrove LONG_MAX., Anthony PERARD, 2011/11/01
- [Qemu-devel] [PATCH V3 12/13] libxl_qmp: Introduce libxl__qmp_pci_del, Anthony PERARD, 2011/11/01
- [Qemu-devel] [PATCH V3 04/13] libxl: Introduce dm-version xenstore key., Anthony PERARD, 2011/11/01
- [Qemu-devel] [PATCH V3 07/13] libxl_qmp: Always insert a command id in the callback_list., Anthony PERARD, 2011/11/01
- Re: [Qemu-devel] [PATCH V3 00/13] libxl: QMP client improvement + pci passthrougth insert through QMP, Anthony PERARD, 2011/11/01