diff options
| author | Bram Moolenaar <Bram@vim.org> | 2016-01-24 16:49:11 +0100 |
|---|---|---|
| committer | Bram Moolenaar <Bram@vim.org> | 2016-01-24 16:49:11 +0100 |
| commit | fcaaae6b3fdbf3421a1ff95a25ae16d82381c39a (patch) | |
| tree | 897597dbdd77dc91e87b5831a4b5e111adfc75ae /src/json.c | |
| parent | 938ee834d345062cd94f8fdfd54fad0019432a83 (diff) | |
| download | vim-git-fcaaae6b3fdbf3421a1ff95a25ae16d82381c39a.tar.gz | |
patch 7.4.1166v7.4.1166
Problem: Can't encode a Funcref into JSON. jsonencode() doesn't handle the
same list or dict twice properly. (Nikolay Pavlov)
Solution: Give an error. Reset copyID when the list or dict is finished.
Diffstat (limited to 'src/json.c')
| -rw-r--r-- | src/json.c | 26 |
1 files changed, 19 insertions, 7 deletions
diff --git a/src/json.c b/src/json.c index e4473830a..da8beea87 100644 --- a/src/json.c +++ b/src/json.c @@ -16,6 +16,7 @@ #include "vim.h" #if defined(FEAT_EVAL) || defined(PROTO) +static int json_encode_item(garray_T *gap, typval_T *val, int copyID); static void json_decode_item(js_read_T *reader, typval_T *res); /* @@ -83,7 +84,11 @@ write_string(garray_T *gap, char_u *str) } } - void +/* + * Encode "val" into "gap". + * Return FAIL or OK. + */ + static int json_encode_item(garray_T *gap, typval_T *val, int copyID) { char_u numbuf[NUMBUFLEN]; @@ -94,7 +99,7 @@ json_encode_item(garray_T *gap, typval_T *val, int copyID) switch (val->v_type) { case VAR_SPECIAL: - switch(val->vval.v_number) + switch (val->vval.v_number) { case VVAL_FALSE: ga_concat(gap, (char_u *)"false"); break; case VVAL_TRUE: ga_concat(gap, (char_u *)"true"); break; @@ -115,8 +120,9 @@ json_encode_item(garray_T *gap, typval_T *val, int copyID) break; case VAR_FUNC: - /* no JSON equivalent, skip */ - break; + /* no JSON equivalent */ + EMSG(_(e_invarg)); + return FAIL; case VAR_LIST: l = val->vval.v_list; @@ -134,12 +140,14 @@ json_encode_item(garray_T *gap, typval_T *val, int copyID) ga_append(gap, '['); for (li = l->lv_first; li != NULL && !got_int; ) { - json_encode_item(gap, &li->li_tv, copyID); + if (json_encode_item(gap, &li->li_tv, copyID) == FAIL) + return FAIL; li = li->li_next; if (li != NULL) ga_append(gap, ','); } ga_append(gap, ']'); + l->lv_copyID = 0; } } break; @@ -172,10 +180,12 @@ json_encode_item(garray_T *gap, typval_T *val, int copyID) ga_append(gap, ','); write_string(gap, hi->hi_key); ga_append(gap, ':'); - json_encode_item(gap, &dict_lookup(hi)->di_tv, - copyID); + if (json_encode_item(gap, &dict_lookup(hi)->di_tv, + copyID) == FAIL) + return FAIL; } ga_append(gap, '}'); + d->dv_copyID = 0; } } break; @@ -187,7 +197,9 @@ json_encode_item(garray_T *gap, typval_T *val, int copyID) break; #endif default: EMSG2(_(e_intern2), "json_encode_item()"); break; + return FAIL; } + return OK; } /* |