Skip to content

Fix: buffer overflow vulnerability in Json::parseString #20846

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: v4
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
6 changes: 3 additions & 3 deletions .appveyor.yml
Original file line number Diff line number Diff line change
@@ -8,9 +8,9 @@ environment:
PYTHON_VERSION: "2.7.13"
PYTHON_ARCH: "32"
matrix:
- build_type: windows32_cmake_test
- build_type: android_cpp_tests
- build_type: android_lua_tests
# - build_type: windows32_cmake_test
# - build_type: android_cpp_tests
# - build_type: android_lua_tests
# - build_type: android_cocos_new_test
# - build_type: android_cpp_empty_test
# - build_type: android_gen_libs
152 changes: 76 additions & 76 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
@@ -12,27 +12,27 @@ on:
workflow_dispatch:

jobs:
ubuntu-18_04:
runs-on: ubuntu-18.04
steps:
- uses: actions/checkout@v2
with:
submodules: recursive
- run: python download-deps.py --r no
- run: echo -e "y" | bash install-deps-linux.sh
- run: cmake -B b -S .
- run: cmake --build b
# ubuntu-18_04:
# runs-on: ubuntu-18.04
# steps:
# - uses: actions/checkout@v2
# with:
# submodules: recursive
# - run: python download-deps.py --r no
# - run: echo -e "y" | bash install-deps-linux.sh
# - run: cmake -B b -S .
# - run: cmake --build b

ubuntu-20_04:
runs-on: ubuntu-20.04
steps:
- uses: actions/checkout@v2
with:
submodules: recursive
- run: python download-deps.py --r no
- run: echo -e "y" | bash install-deps-linux.sh
- run: cmake -B b -S .
- run: cmake --build b
# ubuntu-20_04:
# runs-on: ubuntu-20.04
# steps:
# - uses: actions/checkout@v2
# with:
# submodules: recursive
# - run: python download-deps.py --r no
# - run: echo -e "y" | bash install-deps-linux.sh
# - run: cmake -B b -S .
# - run: cmake --build b

windows-2019:
runs-on: windows-2019
@@ -54,64 +54,64 @@ jobs:
- run: cmake -B b -S . -G "Visual Studio 17 2022" -A Win32
- run: cmake --build b

macos-10_15:
runs-on: macos-10.15
steps:
- uses: actions/checkout@v2
with:
submodules: recursive
- run: python download-deps.py --r no
- run: cmake -B b -S . -GXcode
- run: cmake --build b
# macos-10_15:
# runs-on: macos-10.15
# steps:
# - uses: actions/checkout@v2
# with:
# submodules: recursive
# - run: python download-deps.py --r no
# - run: cmake -B b -S . -GXcode
# - run: cmake --build b

macos-11:
runs-on: macos-11
steps:
- uses: actions/checkout@v2
with:
submodules: recursive
- run: python download-deps.py --r no
- run: cmake -B b -S . -GXcode
- run: cmake --build b
# macos-11:
# runs-on: macos-11
# steps:
# - uses: actions/checkout@v2
# with:
# submodules: recursive
# - run: python download-deps.py --r no
# - run: cmake -B b -S . -GXcode
# - run: cmake --build b

macos-10_15_ios:
runs-on: macos-10.15
steps:
- uses: actions/checkout@v2
with:
submodules: recursive
- run: python download-deps.py --r no
- run: cmake -B b -S . -GXcode -DCMAKE_SYSTEM_NAME=iOS -DCMAKE_OSX_SYSROOT=iphonesimulator
- run: cmake --build b --config Release --target cpp-tests -- -quiet
# macos-10_15_ios:
# runs-on: macos-10.15
# steps:
# - uses: actions/checkout@v2
# with:
# submodules: recursive
# - run: python download-deps.py --r no
# - run: cmake -B b -S . -GXcode -DCMAKE_SYSTEM_NAME=iOS -DCMAKE_OSX_SYSROOT=iphonesimulator
# - run: cmake --build b --config Release --target cpp-tests -- -quiet

macos-11_ios:
runs-on: macos-11
steps:
- uses: actions/checkout@v2
with:
submodules: recursive
- run: python download-deps.py --r no
- run: cmake -B b -S . -GXcode -DCMAKE_SYSTEM_NAME=iOS -DCMAKE_OSX_SYSROOT=iphonesimulator
- run: cmake --build b --config Release --target cpp-tests -- -quiet -destination "platform=iOS Simulator,name=iPhone Retina (4-inch)"
# macos-11_ios:
# runs-on: macos-11
# steps:
# - uses: actions/checkout@v2
# with:
# submodules: recursive
# - run: python download-deps.py --r no
# - run: cmake -B b -S . -GXcode -DCMAKE_SYSTEM_NAME=iOS -DCMAKE_OSX_SYSROOT=iphonesimulator
# - run: cmake --build b --config Release --target cpp-tests -- -quiet -destination "platform=iOS Simulator,name=iPhone Retina (4-inch)"

windows-2019-android:
runs-on: windows-2019
steps:
- uses: actions/checkout@v2
with:
submodules: recursive
- run: python download-deps.py --r no
- run: ./gradlew assembleRelease -PPROP_BUILD_TYPE=cmake --info
shell: bash
working-directory: tests/cpp-tests/proj.android
# windows-2019-android:
# runs-on: windows-2019
# steps:
# - uses: actions/checkout@v2
# with:
# submodules: recursive
# - run: python download-deps.py --r no
# - run: ./gradlew assembleRelease -PPROP_BUILD_TYPE=cmake --info
# shell: bash
# working-directory: tests/cpp-tests/proj.android

ubuntu-20_04-android:
runs-on: ubuntu-20.04
steps:
- uses: actions/checkout@v2
with:
submodules: recursive
- run: python download-deps.py --r no
- run: ./gradlew assembleRelease -PPROP_BUILD_TYPE=cmake --info
shell: bash
working-directory: tests/cpp-tests/proj.android
# ubuntu-20_04-android:
# runs-on: ubuntu-20.04
# steps:
# - uses: actions/checkout@v2
# with:
# submodules: recursive
# - run: python download-deps.py --r no
# - run: ./gradlew assembleRelease -PPROP_BUILD_TYPE=cmake --info
# shell: bash
# working-directory: tests/cpp-tests/proj.android
6 changes: 3 additions & 3 deletions .gitmodules
Original file line number Diff line number Diff line change
@@ -1,9 +1,9 @@
[submodule "tools/cocos2d-console"]
path = tools/cocos2d-console
url = git://github.com/cocos2d/cocos2d-console.git
url = https://github.com/cocos2d/cocos2d-console.git
[submodule "tools/bindings-generator"]
path = tools/bindings-generator
url = git://github.com/cocos2d/bindings-generator.git
url = https://github.com/cocos2d/bindings-generator.git
[submodule "tests/cpp-tests/Resources/ccs-res"]
path = tests/cpp-tests/Resources/ccs-res
url = git://github.com/dumganhar/ccs-res.git
url = https://github.com/dumganhar/ccs-res.git
2 changes: 1 addition & 1 deletion .travis.yml
Original file line number Diff line number Diff line change
@@ -29,7 +29,7 @@ matrix:
language: android
sudo: required
dist: xenial
# android_lua cmake
android_lua cmake
- os: linux
env: BUILD_TARGET=android_lua_cmake
language: android
10 changes: 10 additions & 0 deletions cmake/Modules/CocosConfigDefine.cmake
Original file line number Diff line number Diff line change
@@ -10,11 +10,14 @@ endif()
#IOS = iOS
#MACOSX = MacOS X
#LINUX = Linux
#OHOS = OHOS
if(${CMAKE_SYSTEM_NAME} MATCHES "Windows")
set(WINDOWS TRUE)
set(PLATFORM_FOLDER win32)
elseif(${CMAKE_SYSTEM_NAME} MATCHES "Android")
set(PLATFORM_FOLDER android)
elseif(CMAKE_SYSTEM_NAME MATCHES "OHOS")
set(PLATFORM_FOLDER ohos)
elseif(${CMAKE_SYSTEM_NAME} MATCHES "Linux")
if(ANDROID)
set(PLATFORM_FOLDER android)
@@ -60,7 +63,11 @@ define_property(TARGET
# check c++ standard
set(CMAKE_C_STANDARD 99)
set(CMAKE_C_STANDARD_REQUIRED ON)
if(OHOS)
set(CMAKE_CXX_STANDARD 14)
else()
set(CMAKE_CXX_STANDARD 11)
endif()
set(CMAKE_CXX_STANDARD_REQUIRED ON)
set(CMAKE_CXX_EXTENSIONS OFF)

@@ -92,6 +99,9 @@ endif()
elseif(ANDROID)
target_compile_definitions(${target} PUBLIC ANDROID)
target_compile_definitions(${target} PUBLIC USE_FILE32API)
elseif(OHOS)
target_compile_definitions(${target} PUBLIC OHOS)
target_compile_definitions(${target} PUBLIC USE_FILE32API)
elseif(WINDOWS)
target_compile_definitions(${target}
PUBLIC WIN32
2 changes: 2 additions & 0 deletions cmake/Modules/CocosConfigDepend.cmake
Original file line number Diff line number Diff line change
@@ -21,6 +21,8 @@ macro(cocos2dx_depend)
set(THREADS_LIBRARIES ${CMAKE_THREAD_LIBS_INIT})
elseif(ANDROID)
list(APPEND PLATFORM_SPECIFIC_LIBS GLESv2 EGL log android OpenSLES)
elseif(OHOS)
list(APPEND PLATFORM_SPECIFIC_LIBS native_drawing EGL GLESv3 hilog_ndk.z ace_ndk.z ace_napi.z uv rawfile.z OpenSLES)
elseif(APPLE)

include_directories(/System/Library/Frameworks)
4 changes: 4 additions & 0 deletions cocos/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -118,6 +118,10 @@ use_cocos2dx_compile_define(cocos2d)
use_cocos2dx_compile_options(cocos2d)

# use all platform related system libs
if (OHOS)

target_link_libraries(cocos2d ${Drawing-lib} ${libace-lib} ${GLES-lib} ${libnapi-lib} ${libuv-lib} ${rawfile-lib} ${EGL-lib} ${hilog-lib} libohaudio.so libavplayer.so libnative_window.so libnative_buffer.so)
endif()
use_cocos2dx_libs_depend(cocos2d)

target_include_directories(cocos2d
2 changes: 2 additions & 0 deletions cocos/audio/AudioEngine.cpp
Original file line number Diff line number Diff line change
@@ -39,6 +39,8 @@
#include "audio/win32/AudioEngine-win32.h"
#elif CC_TARGET_PLATFORM == CC_PLATFORM_LINUX
#include "audio/linux/AudioEngine-linux.h"
#elif CC_TARGET_PLATFORM == CC_PLATFORM_OHOS
#include "audio/ohos/AudioEngine-inl.h"
#endif

#define TIME_DELAY_PRECISION 0.0001
72 changes: 72 additions & 0 deletions cocos/audio/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -105,6 +105,78 @@ elseif(ANDROID)
audio/android/tinysndfile.cpp
)

elseif(OHOS)
set(COCOS_AUDIO_PLATFORM_HEADER
audio/ohos/PcmAudioService.h
audio/ohos/AudioBufferProvider.h
audio/ohos/IAudioPlayer.h
audio/ohos/AudioResampler.h
audio/ohos/AudioDecoder.h
audio/ohos/AudioResamplerPublic.h
audio/ohos/AudioMixer.h
audio/ohos/tinysndfile.h
audio/ohos/mp3reader.h
audio/ohos/AudioMixerOps.h
audio/ohos/cutils/bitops.h
audio/ohos/cutils/log.h
audio/ohos/audio.h
audio/ohos/AudioPlayerProvider.h
audio/ohos/utils/Utils.h
audio/ohos/utils/Errors.h
audio/ohos/utils/Compat.h
audio/ohos/AudioDecoderOgg.h
audio/ohos/Track.h
audio/ohos/OpenSLHelper.h
audio/ohos/PcmAudioPlayer.h
audio/ohos/AssetFd.h
audio/ohos/PcmBufferProvider.h
audio/ohos/CCThreadPool.h
audio/ohos/audio_utils/include/audio_utils/minifloat.h
audio/ohos/audio_utils/include/audio_utils/primitives.h
audio/ohos/audio_utils/AudioDef.h
audio/ohos/audio_utils/RefCounted.h
audio/ohos/ICallerThreadUtils.h
audio/ohos/AudioDecoderWav.h
audio/ohos/AudioDecoderProvider.h
audio/ohos/UrlAudioPlayer.h
audio/ohos/AudioDecoderSLES.h
audio/ohos/AudioDecoderMp3.h
audio/ohos/PcmData.h
audio/ohos/AudioMixerController.h
audio/ohos/AudioResamplerCubic.h
audio/ohos/AudioEngine-inl.h
audio/ohos/IVolumeProvider.h
audio/ohos/Macros.h
)

set(COCOS_AUDIO_PLATFORM_SRC
audio/ohos/AudioEngine-inl.cpp
audio/ohos/CCThreadPool.cpp
audio/ohos/AssetFd.cpp
audio/ohos/AudioDecoder.cpp
audio/ohos/AudioDecoderProvider.cpp
audio/ohos/AudioDecoderSLES.cpp
audio/ohos/AudioDecoderOgg.cpp
audio/ohos/AudioDecoderMp3.cpp
audio/ohos/AudioDecoderWav.cpp
audio/ohos/AudioPlayerProvider.cpp
audio/ohos/AudioResampler.cpp
audio/ohos/AudioResamplerCubic.cpp
audio/ohos/PcmBufferProvider.cpp
audio/ohos/PcmAudioPlayer.cpp
audio/ohos/PcmData.cpp
audio/ohos/PcmAudioService.cpp
audio/ohos/UrlAudioPlayer.cpp
audio/ohos/AudioMixerController.cpp
audio/ohos/AudioMixer.cpp
audio/ohos/mp3reader.cpp
audio/ohos/tinysndfile.cpp
audio/ohos/Track.cpp
audio/ohos/audio_utils/RefCounted.cpp
audio/ohos/audio_utils/minifloat.cpp
audio/ohos/audio_utils/primitives.cpp
audio/ohos/utils/Utils.cpp
)
elseif(LINUX)
set(COCOS_AUDIO_PLATFORM_HEADER
audio/linux/AudioEngine-linux.h
45 changes: 45 additions & 0 deletions cocos/audio/ohos/AssetFd.cpp
Original file line number Diff line number Diff line change
@@ -0,0 +1,45 @@
/****************************************************************************
Copyright (c) 2016 Chukong Technologies Inc.
Copyright (c) 2017-2018 Xiamen Yaji Software Co., Ltd.

http://www.cocos2d-x.org

Permission is hereby granted, free of charge, to any person obtaining a copy
of this software and associated documentation files (the "Software"), to deal
in the Software without restriction, including without limitation the rights
to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
copies of the Software, and to permit persons to whom the Software is
furnished to do so, subject to the following conditions:

The above copyright notice and this permission notice shall be included in
all copies or substantial portions of the Software.

THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
THE SOFTWARE.
****************************************************************************/

#define LOG_TAG "AssetFd"

#include "cutils/log.h"
#include "AssetFd.h"

namespace cocos2d {

AssetFd::AssetFd(int assetFd)
: _assetFd(assetFd) {
}

AssetFd::~AssetFd() {
ALOGV("~AssetFd: %d", _assetFd);
if (_assetFd > 0) {
::close(_assetFd);
_assetFd = 0;
}
};

} // namespace CocosDenshion
Loading