From 7c5fa6de50bf46a21571770a3b23cf62eadc82d1 Mon Sep 17 00:00:00 2001 From: Emanuele Torre Date: Wed, 13 Dec 2023 00:48:15 +0100 Subject: [PATCH] jv_parse: let decNumberFromString/strtod parse complex nans as a NaN Before this patch (when using decNumber), "Nan123" was parsed as a NaN, only if the first n was uppercase. --- src/jv_parse.c | 2 +- tests/jq.test | 5 +++++ 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/src/jv_parse.c b/src/jv_parse.c index 110483a4af..9755b8ac47 100644 --- a/src/jv_parse.c +++ b/src/jv_parse.c @@ -514,7 +514,7 @@ static pfunc check_literal(struct jv_parser* p) { case 'f': pattern = "false"; plen = 5; v = jv_false(); break; case 'n': // if it starts with 'n', it could be a literal "nan" - if (p->tokenpos != 3) { + if (p->tokenbuf[1] == 'u') { pattern = "null"; plen = 4; v = jv_null(); } } diff --git a/tests/jq.test b/tests/jq.test index 945ea04f1c..39e3c91191 100644 --- a/tests/jq.test +++ b/tests/jq.test @@ -1938,6 +1938,11 @@ tojson | fromjson {"a":nan} {"a":null} +# also "nan with payload" #2985 +fromjson | isnan +"nan1234" +true + # calling input/0, or debug/0 in a test doesn't crash jq