Skip to content

Conversation

@zhuizhuhaomeng
Copy link

  • Fix integer overflows due to using wrong integer size.
  • Add assertions / panic when overflow still happens.

@zhuizhuhaomeng zhuizhuhaomeng changed the title Lua cjson and cmsgpack integer overflow issues (CVE-2022-24834) Bugfix: Lua cjson and cmsgpack integer overflow issues (CVE-2022-24834) Jul 18, 2023
@zhuizhuhaomeng zhuizhuhaomeng changed the title Bugfix: Lua cjson and cmsgpack integer overflow issues (CVE-2022-24834) WIP Bugfix: Lua cjson and cmsgpack integer overflow issues (CVE-2022-24834) Jul 18, 2023
* Fix integer overflows due to using wrong integer size.
* Add assertions / panic when overflow still happens.

Co-authored-by: Yossi Gottlieb <yossigo@gmail.com>
@zhuizhuhaomeng zhuizhuhaomeng changed the title WIP Bugfix: Lua cjson and cmsgpack integer overflow issues (CVE-2022-24834) Bugfix: Lua cjson and cmsgpack integer overflow issues (CVE-2022-24834) Jul 18, 2023
@zhuizhuhaomeng zhuizhuhaomeng merged commit 2bfad8f into master Jul 20, 2023
windmgc pushed a commit to Kong/kong that referenced this pull request Jul 26, 2023
Patch lua-cjson to make it can process a large string correctly.

OpenResty PR: openresty/lua-cjson#94

KAG-2151
team-gateway-bot pushed a commit to Kong/kong that referenced this pull request Jul 26, 2023
Patch lua-cjson to make it can process a large string correctly.

OpenResty PR: openresty/lua-cjson#94

KAG-2151

(cherry picked from commit 65de2c3)
windmgc pushed a commit to Kong/kong that referenced this pull request Jul 26, 2023
Patch lua-cjson to make it can process a large string correctly.

OpenResty PR: openresty/lua-cjson#94

KAG-2151

(cherry picked from commit 65de2c3)
team-gateway-bot pushed a commit to Kong/kong that referenced this pull request Jul 27, 2023
Patch lua-cjson to make it can process a large string correctly.

OpenResty PR: openresty/lua-cjson#94

KAG-2151

(cherry picked from commit 65de2c3)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants