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
Show file tree
Hide file tree
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
Expand Up @@ -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
Expand Down
152 changes: 76 additions & 76 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand All @@ -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
Expand Up @@ -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
Expand Down
10 changes: 10 additions & 0 deletions cmake/Modules/CocosConfigDefine.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -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)
Expand Down Expand Up @@ -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)

Expand Down Expand Up @@ -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
Expand Down
2 changes: 2 additions & 0 deletions cmake/Modules/CocosConfigDepend.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -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)
Expand Down
4 changes: 4 additions & 0 deletions cocos/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down
2 changes: 2 additions & 0 deletions cocos/audio/AudioEngine.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down
72 changes: 72 additions & 0 deletions cocos/audio/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down
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