diff options
| author | Pascal Quantin <pascal@wireshark.org> | 2020-08-14 14:49:48 +0200 |
|---|---|---|
| committer | Pascal Quantin <pascal@wireshark.org> | 2020-08-14 13:53:16 +0000 |
| commit | 62c4e2525f23d200b39caa7ac14080d943024728 (patch) | |
| tree | 63a9ce621ae2154908b8df26cfca4e6280ce2f76 /epan/dissectors/asn1 | |
| parent | ee42cb79243f69f35a24639ba3f092a6d44a83ec (diff) | |
Check that at least one token exists to consider the JSON as valid
Bug: 16780
Change-Id: I07ca12675fc79a7c524719d18b85e5d3dada6652
Reviewed-on: https://code.wireshark.org/review/38160
Petri-Dish: Pascal Quantin <pascal@wireshark.org>
Tested-by: Petri Dish Buildbot
Reviewed-by: Pascal Quantin <pascal@wireshark.org>
Diffstat (limited to 'epan/dissectors/asn1')
| -rw-r--r-- | epan/dissectors/asn1/ngap/packet-ngap-template.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/epan/dissectors/asn1/ngap/packet-ngap-template.c b/epan/dissectors/asn1/ngap/packet-ngap-template.c index 8f2dc94412..1c91c199a8 100644 --- a/epan/dissectors/asn1/ngap/packet-ngap-template.c +++ b/epan/dissectors/asn1/ngap/packet-ngap-template.c @@ -470,10 +470,10 @@ dissect_ngap_media_type(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, voi json_data = tvb_get_string_enc(wmem_packet_scope(), json_tvb, 0, tvb_reported_length(json_tvb), ENC_UTF_8|ENC_NA); ret = json_parse(json_data, NULL, 0); - if (ret < 0) + if (ret <= 0) return 0; tokens = wmem_alloc_array(wmem_packet_scope(), jsmntok_t, ret); - if (json_parse(json_data, tokens, ret) < 0) + if (json_parse(json_data, tokens, ret) <= 0) return 0; cur_tok = json_get_object(json_data, tokens, "n2InfoContainer"); if (!cur_tok) { |