diff --git a/common/src/daudio_latency_test.cpp b/common/src/daudio_latency_test.cpp index d88024b95ae6b8f656a77bf4e3b3f4785b99d10b..ab420fd0750a1c63671bfaa3f2a166af3a0eb292 100644 --- a/common/src/daudio_latency_test.cpp +++ b/common/src/daudio_latency_test.cpp @@ -24,10 +24,11 @@ #undef DH_LOG_TAG #define DH_LOG_TAG "DAudioLatencyTest" - namespace OHOS { namespace DistributedHardware { IMPLEMENT_SINGLE_INSTANCE(DAudioLatencyTest); +constexpr int32_t MAXSIZE = 8192; + DAudioLatencyTest::DAudioLatencyTest() { DHLOGI("DAudioLatencyTest constructed."); @@ -69,6 +70,10 @@ int32_t DAudioLatencyTest::AddRecordTime(const int64_t recordBeepTime) bool DAudioLatencyTest::IsFrameHigh(const int16_t *audioData, const int32_t size, int32_t threshhold) { + if (size > MAXSIZE) { + DHLOGI("size=%{public}d is over range", size); + return false; + } int32_t max = 0; for (int32_t i = 0; i < size; i++) { int16_t f = abs(audioData[i]); diff --git a/common/test/unittest/src/daudio_utils_test.cpp b/common/test/unittest/src/daudio_utils_test.cpp index 37b60bc9964e467b120f1edd78e2492e207f8e68..7c2eb220461fa034c4fae8f4ff14b94af8ba6c23 100644 --- a/common/test/unittest/src/daudio_utils_test.cpp +++ b/common/test/unittest/src/daudio_utils_test.cpp @@ -164,6 +164,7 @@ HWTEST_F(DAudioUtilsTest, DAudioUtilTest_002, TestSize.Level1) cJSON_AddStringToObject(jsonObj, "one", "one"); cJSON_AddNumberToObject(jsonObj, "two", 2); CJsonParamCheck(jsonObj, keys); + cJSON_Delete(jsonObj); int64_t tvSec; int64_t tvNSec; diff --git a/interfaces/inner_kits/native_cpp/test/fuzztest/sinkproxyinitsink_fuzzer/sinkproxyinitsink_fuzzer.cpp b/interfaces/inner_kits/native_cpp/test/fuzztest/sinkproxyinitsink_fuzzer/sinkproxyinitsink_fuzzer.cpp index 465eeb8031fca606d6075b034af4249814b3b1d3..84fc5dcfb54eb7bb2fc6a7515815e2c36af7cb76 100644 --- a/interfaces/inner_kits/native_cpp/test/fuzztest/sinkproxyinitsink_fuzzer/sinkproxyinitsink_fuzzer.cpp +++ b/interfaces/inner_kits/native_cpp/test/fuzztest/sinkproxyinitsink_fuzzer/sinkproxyinitsink_fuzzer.cpp @@ -41,7 +41,7 @@ void SinkProxyInitSinkFuzzTest(const uint8_t* data, size_t size) return; } std::shared_ptr dAudioProxy = std::make_shared(remoteObject); - sptr dAudioSinkIpcCallback = new DAudioSinkIpcCallback(); + sptr dAudioSinkIpcCallback(new DAudioSinkIpcCallback()); dAudioProxy->InitSink(params, dAudioSinkIpcCallback); } } diff --git a/interfaces/inner_kits/native_cpp/test/fuzztest/sourceproxyinitsource_fuzzer/sourceproxyinitsource_fuzzer.cpp b/interfaces/inner_kits/native_cpp/test/fuzztest/sourceproxyinitsource_fuzzer/sourceproxyinitsource_fuzzer.cpp index 18edf117f69a5bb1f83161a715aefa315a7389b5..0ca487bdac62aa6832d8f0970f26da21695e65f8 100644 --- a/interfaces/inner_kits/native_cpp/test/fuzztest/sourceproxyinitsource_fuzzer/sourceproxyinitsource_fuzzer.cpp +++ b/interfaces/inner_kits/native_cpp/test/fuzztest/sourceproxyinitsource_fuzzer/sourceproxyinitsource_fuzzer.cpp @@ -42,8 +42,8 @@ void SourceProxyInitSourceFuzzTest(const uint8_t* data, size_t size) return; } std::shared_ptr dAudioProxy = std::make_shared(remoteObject); - sptr callback = new DAudioIpcCallback(); - + sptr callback(new DAudioIpcCallback()); + dAudioProxy->InitSource(params, callback); } }