diff options
author | Christopher Watford <christopher.watford@gmail.com> | 2009-07-08 03:46:10 +0000 |
---|---|---|
committer | Christopher Watford <christopher.watford@gmail.com> | 2009-07-08 03:46:10 +0000 |
commit | 543bb1431f12cd62e7d9d4b65989919409e239c9 (patch) | |
tree | a1a4d2a7f111fe9058243418845153eef353eefd | |
parent | c5cbf8214ab628bead203d2ac3a1948bc3005801 (diff) | |
download | json-c-543bb1431f12cd62e7d9d4b65989919409e239c9.tar.gz |
Erik Hovland (3):
Fix any noticeable spelling or grammar errors.
Make sure every va_start has a va_end.
Check all pointers for validity.
git-svn-id: http://svn.metaparadigm.com/svn/json-c/trunk@41 327403b1-1117-474d-bef2-5cb71233fd97
-rw-r--r-- | ChangeLog | 4 | ||||
-rw-r--r-- | json_tokener.c | 1 | ||||
-rw-r--r-- | printbuf.c | 4 |
3 files changed, 7 insertions, 2 deletions
@@ -1,4 +1,8 @@ 0.9 + * Fix any noticeable spelling or grammar errors. + * Make sure every va_start has a va_end. + * Check all pointers for validity. + Erik Hovland, erik at hovland dot org * Fix json_object_get_boolean to return false for empty string Spotted by Vitaly Kruglikov, Vitaly dot Kruglikov at palm dot com * optimizations to json_tokener_parse_ex(), printbuf_memappend() diff --git a/json_tokener.c b/json_tokener.c index 8e8c6d9..a1951b9 100644 --- a/json_tokener.c +++ b/json_tokener.c @@ -64,6 +64,7 @@ struct json_tokener* json_tokener_new(void) struct json_tokener *tok; tok = (struct json_tokener*)calloc(1, sizeof(struct json_tokener)); + if (!tok) return NULL; tok->pb = printbuf_new(); json_tokener_reset(tok); return tok; @@ -120,10 +120,10 @@ int sprintbuf(struct printbuf *p, const char *msg, ...) /* if string is greater than stack buffer, then use dynamic string with vasprintf. Note: some implementation of vsnprintf return -1 if output is truncated whereas some return the number of bytes that - would have been writen - this code handles both cases. */ + would have been written - this code handles both cases. */ if(size == -1 || size > 127) { va_start(ap, msg); - if((size = vasprintf(&t, msg, ap)) == -1) return -1; + if((size = vasprintf(&t, msg, ap)) == -1) { va_end(ap); return -1; } va_end(ap); printbuf_memappend(p, t, size); free(t); |