mirror/ultrajson
1
0
Fork 0
mirror of https://github.com/ultrajson/ultrajson.git synced 2024-11-27 12:14:43 +01:00
Code Issues

Decoder now properly handles detection of trailing commas in arrays and trailing data at end of input

Browse Source
This commit is contained in:
Jonas Tarnstrom 2012-08-14 11:24:08 +02:00
parent 684a4ab2f6
commit 49aa2c8e56
1 changed files with 35 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

40
lib/ultrajsondec.c
View File

@ -67,6 +67,11 @@ static JSOBJ SetError( struct DecoderState *ds, int offset, const char *message)
return NULL;
}
static void ClearError( struct DecoderState *ds)
{
ds->dec->errorOffset = 0;
ds->dec->errorStr = NULL;
}
FASTCALL_ATTR JSOBJ FASTCALL_MSVC decode_numeric ( struct DecoderState *ds)
{
@ -637,24 +642,38 @@ FASTCALL_ATTR JSOBJ FASTCALL_MSVC decode_array( struct DecoderState *ds)
{
JSOBJ itemValue;
JSOBJ newObj = ds->dec->newArray();
int len = 0;
ds->lastType = JT_INVALID;
ds->start ++;
while (1)//(*ds->start) != '\0')
{
SkipWhitespace(ds);
if ((*ds->start) == ']')
{
ds->start++;
return newObj;
}
itemValue = decode_any(ds);
if (itemValue == NULL)
{
switch (*(ds->start++))
{
case ']':
if (len > 0)
{
ds->dec->releaseObject(newObj);
return SetError(ds, -1, "Unexpected trailing comma in array");
}
// We end up here when decoding empty lists "[]"
ClearError(ds);
return newObj;
default:
ds->dec->releaseObject(newObj);
return SetError(ds, -1, "Unexpected character in found when decoding array value");
}
ds->dec->releaseObject(newObj);
return NULL;
}
@ -675,6 +694,8 @@ FASTCALL_ATTR JSOBJ FASTCALL_MSVC decode_array( struct DecoderState *ds)
ds->dec->releaseObject(newObj);
return SetError(ds, -1, "Unexpected character in found when decoding array value");
}
len ++;
}
ds->dec->releaseObject(newObj);
@ -828,5 +849,12 @@ JSOBJ JSON_DecodeObject(JSONObjectDecoder *dec, const char *buffer, size_t cbBuf
{
dec->free(ds.escStart);
}
if (ds.start != ds.end && ret)
{
dec->releaseObject(ret);
return SetError(&ds, -1, "Trailing data");
}
return ret;
}