Compare commits
No commits in common. "c804261b12a2045090becc4758eb52201785f762" and "2eff54f282bc0a29a8dcd168487c31f2eaa02ac2" have entirely different histories.
c804261b12
...
2eff54f282
@ -1,59 +0,0 @@
|
|||||||
From 8269bc2bc289e9d343bae51cdf6d23ef0950e001 Mon Sep 17 00:00:00 2001
|
|
||||||
From: Florin Malita <fmalita@gmail.com>
|
|
||||||
Date: Tue, 15 May 2018 22:48:07 -0400
|
|
||||||
Subject: [PATCH] Prevent int underflow when parsing exponents
|
|
||||||
|
|
||||||
When parsing negative exponents, the current implementation takes
|
|
||||||
precautions for |exp| to not underflow int.
|
|
||||||
|
|
||||||
But that is not sufficient: later on [1], |exp + expFrac| is also
|
|
||||||
stored to an int - so we must ensure that the sum stays within int
|
|
||||||
representable values.
|
|
||||||
|
|
||||||
Update the exp clamping logic to take expFrac into account.
|
|
||||||
|
|
||||||
[1] https://github.com/Tencent/rapidjson/blob/master/include/rapidjson/reader.h#L1690
|
|
||||||
---
|
|
||||||
include/rapidjson/reader.h | 11 ++++++++++-
|
|
||||||
test/unittest/readertest.cpp | 1 +
|
|
||||||
2 files changed, 11 insertions(+), 1 deletion(-)
|
|
||||||
|
|
||||||
diff --git a/include/rapidjson/reader.h b/include/rapidjson/reader.h
|
|
||||||
index 7441eda4..f95aef42 100644
|
|
||||||
--- a/include/rapidjson/reader.h
|
|
||||||
+++ b/include/rapidjson/reader.h
|
|
||||||
@@ -1632,9 +1632,18 @@ private:
|
|
||||||
if (RAPIDJSON_LIKELY(s.Peek() >= '0' && s.Peek() <= '9')) {
|
|
||||||
exp = static_cast<int>(s.Take() - '0');
|
|
||||||
if (expMinus) {
|
|
||||||
+ // (exp + expFrac) must not underflow int => we're detecting when -exp gets
|
|
||||||
+ // dangerously close to INT_MIN (a pessimistic next digit 9 would push it into
|
|
||||||
+ // underflow territory):
|
|
||||||
+ //
|
|
||||||
+ // -(exp * 10 + 9) + expFrac >= INT_MIN
|
|
||||||
+ // <=> exp <= (expFrac - INT_MIN - 9) / 10
|
|
||||||
+ RAPIDJSON_ASSERT(expFrac <= 0);
|
|
||||||
+ int maxExp = (expFrac + 2147483639) / 10;
|
|
||||||
+
|
|
||||||
while (RAPIDJSON_LIKELY(s.Peek() >= '0' && s.Peek() <= '9')) {
|
|
||||||
exp = exp * 10 + static_cast<int>(s.Take() - '0');
|
|
||||||
- if (exp >= 214748364) { // Issue #313: prevent overflow exponent
|
|
||||||
+ if (RAPIDJSON_UNLIKELY(exp > maxExp)) {
|
|
||||||
while (RAPIDJSON_UNLIKELY(s.Peek() >= '0' && s.Peek() <= '9')) // Consume the rest of exponent
|
|
||||||
s.Take();
|
|
||||||
}
|
|
||||||
diff --git a/test/unittest/readertest.cpp b/test/unittest/readertest.cpp
|
|
||||||
index e5308019..c4800b93 100644
|
|
||||||
--- a/test/unittest/readertest.cpp
|
|
||||||
+++ b/test/unittest/readertest.cpp
|
|
||||||
@@ -242,6 +242,7 @@ static void TestParseDouble() {
|
|
||||||
TEST_DOUBLE(fullPrecision, "1e-214748363", 0.0); // Maximum supported negative exponent
|
|
||||||
TEST_DOUBLE(fullPrecision, "1e-214748364", 0.0);
|
|
||||||
TEST_DOUBLE(fullPrecision, "1e-21474836311", 0.0);
|
|
||||||
+ TEST_DOUBLE(fullPrecision, "1.00000000001e-2147483638", 0.0);
|
|
||||||
TEST_DOUBLE(fullPrecision, "0.017976931348623157e+310", 1.7976931348623157e+308); // Max double in another form
|
|
||||||
|
|
||||||
// Since
|
|
||||||
--
|
|
||||||
2.20.1
|
|
||||||
|
|
||||||
@ -1,17 +1,13 @@
|
|||||||
%global debug_package %{nil}
|
%global debug_package %{nil}
|
||||||
Name: rapidjson
|
Name: rapidjson
|
||||||
Version: 1.1.0
|
Version: 1.1.0
|
||||||
Release: 13
|
Release: 10
|
||||||
Summary: small & selft-contained fast JSON parser and generator for C++
|
Summary: small & selft-contained fast JSON parser and generator for C++
|
||||||
License: MIT
|
License: MIT
|
||||||
URL: http://miloyip.github.io/rapidjson
|
URL: http://miloyip.github.io/rapidjson
|
||||||
Source0: https://github.com/miloyip/rapidjson/archive/v%{version}.tar.gz#/rapidjson-%{version}.tar.gz
|
Source0: https://github.com/miloyip/rapidjson/archive/v%{version}.tar.gz#/rapidjson-%{version}.tar.gz
|
||||||
Patch0000: rapidjson-1.1.0-do_not_include_gtest_src_dir.patch
|
Patch0000: rapidjson-1.1.0-do_not_include_gtest_src_dir.patch
|
||||||
Patch0001: backport-CVE-2024-38517.patch
|
BuildRequires: cmake gcc-c++ gtest-devel valgrind
|
||||||
BuildRequires: cmake gcc-c++ gtest-devel
|
|
||||||
%ifarch "%{valgrind_arches}"
|
|
||||||
BuildRequires: valgrind
|
|
||||||
%endif
|
|
||||||
|
|
||||||
%description
|
%description
|
||||||
RapidJSON as a fast JSON parser which generator for c++. It`s inspired by
|
RapidJSON as a fast JSON parser which generator for c++. It`s inspired by
|
||||||
@ -58,7 +54,7 @@ done
|
|||||||
|
|
||||||
%build
|
%build
|
||||||
cd build
|
cd build
|
||||||
%cmake -DDOC_INSTALL_DIR=%{_pkgdocdir} -DRAPIDJSON_BUILD_CXX11:BOOL=OFF -DGTESTSRC_FOUND=TRUE -DGTEST_SOURCE_DIR=. ..
|
%cmake -DDOC_INSTALL_DIR=%{_pkgdocdir} -DGTESTSRC_FOUND=TRUE -DGTEST_SOURCE_DIR=. ..
|
||||||
%make_build
|
%make_build
|
||||||
cd -
|
cd -
|
||||||
|
|
||||||
@ -92,15 +88,6 @@ cd -
|
|||||||
%doc %{_pkgdocdir}
|
%doc %{_pkgdocdir}
|
||||||
|
|
||||||
%changelog
|
%changelog
|
||||||
* Thu Jul 11 2024 zhangxianting <zhangxianting@uniontech.com> - 1.1.0-13
|
|
||||||
- Fix CVE-2024-38517
|
|
||||||
|
|
||||||
* Mon Apr 29 2024 yinsist <jianhui.oerv@isrc.iscas.ac.cn> - 1.1.0-12
|
|
||||||
- Valgrind does not support certain architectures like RISC-V, Before depending on Valgrind, first check if Valgrind supports the architecture
|
|
||||||
|
|
||||||
* Fri Aug 25 2023 liyanan <thistleslyn@163.com> - 1.1.0-11
|
|
||||||
- Do not force C++11: gtest 1.13.0 requires at least C++14
|
|
||||||
|
|
||||||
* Wed Jul 29 2020 lingsheng <lingsheng@huawei.com> - 1.1.0-10
|
* Wed Jul 29 2020 lingsheng <lingsheng@huawei.com> - 1.1.0-10
|
||||||
- Not use hardlink absolute path
|
- Not use hardlink absolute path
|
||||||
|
|
||||||
|
|||||||
@ -1,4 +1,4 @@
|
|||||||
version_control: github
|
version-control: github
|
||||||
src_repo: Tencent/rapidjson
|
src_repo: Tencent/rapidjson
|
||||||
tag_prefix: "^v"
|
tag_prefix: v^
|
||||||
separator: .
|
seperator: .
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user