From 290972f85825ae385209ab3db6cc565d76d47af0 Mon Sep 17 00:00:00 2001 From: panmingxuan Date: Sun, 22 Jun 2025 17:09:50 +0800 Subject: [PATCH] Fix fuzz test fix fuzz test Issue: https://gitee.com/openharmony/arkcompiler_toolchain/issues/ICGX63 Signed-off-by: panmingxuan --- test/fuzztest/BUILD.gn | 4 ++++ .../baseptbase64decode_fuzzer/baseptbase64decode_fuzzer.cpp | 3 +++ .../baseptbase64encode_fuzzer/baseptbase64encode_fuzzer.cpp | 3 +++ .../baseptsbreakpointgetbreakpointid_fuzzer.cpp | 3 +++ .../baseptsbreakpointgetlocation_fuzzer.cpp | 3 +++ .../baseptsbreakpointgetname_fuzzer.cpp | 3 +++ .../baseptspausedgetcallframes_fuzzer.cpp | 3 +++ .../baseptspausedgetdata_fuzzer.cpp | 3 +++ .../baseptspausedgethitbreakpoints_fuzzer.cpp | 3 +++ .../baseptspausedgetname_fuzzer.cpp | 3 +++ .../baseptspausedgetreason_fuzzer.cpp | 3 +++ .../protocolhandlerwait_fuzzer/protocolhandlerwait_fuzzer.cpp | 3 ++- 12 files changed, 36 insertions(+), 1 deletion(-) diff --git a/test/fuzztest/BUILD.gn b/test/fuzztest/BUILD.gn index e79944a8..d44d360d 100644 --- a/test/fuzztest/BUILD.gn +++ b/test/fuzztest/BUILD.gn @@ -32,6 +32,7 @@ group("fuzztest") { "protocolhandlerresponse_fuzzer:fuzztest", "protocolhandlerrun_fuzzer:fuzztest", "protocolhandlersendreply_fuzzer:fuzztest", + "protocolhandlerwait_fuzzer:fuzztest", "pttypesbreakpointparseid_fuzzer:fuzztest", "pttypesbreakpointtostring_fuzzer:fuzztest", "pttypesexceptiondetails_fuzzer:fuzztest", @@ -45,6 +46,8 @@ group("fuzztest") { "pttypesexceptiondetailstext_fuzzer:fuzztest", "pttypesexceptiondetailstojson_fuzzer:fuzztest", "pttypesexceptiondetailsurl_fuzzer:fuzztest", + "pttypesfunctionremoteobject_fuzzer:fuzztest", + "pttypesgeneratorfunctionremoteobject_fuzzer:fuzztest", "pttypesinternalpropertydesp_fuzzer:fuzztest", "pttypesinternalpropertydespcreate_fuzzer:fuzztest", "pttypesinternalpropertydespname_fuzzer:fuzztest", @@ -75,5 +78,6 @@ group("fuzztest") { "pttypesremoteobjectunserializablevalue_fuzzer:fuzztest", "pttypesstringremoteobject_fuzzer:fuzztest", "pttypessymbolremoteobject_fuzzer:fuzztest", + "wsserver:fuzztest", ] } diff --git a/test/fuzztest/base/ptbase64/baseptbase64decode_fuzzer/baseptbase64decode_fuzzer.cpp b/test/fuzztest/base/ptbase64/baseptbase64decode_fuzzer/baseptbase64decode_fuzzer.cpp index 33d44535..809af5d5 100644 --- a/test/fuzztest/base/ptbase64/baseptbase64decode_fuzzer/baseptbase64decode_fuzzer.cpp +++ b/test/fuzztest/base/ptbase64/baseptbase64decode_fuzzer/baseptbase64decode_fuzzer.cpp @@ -33,6 +33,9 @@ namespace OHOS { if (size <= 0) { return; } + if (size > MAXBYTELEN) { + size = MAXBYTELEN; + } auto src = cn->GetString(data, size); std::string dest; dest.resize(PtBase64::EncodedSize(src.size())); diff --git a/test/fuzztest/base/ptbase64/baseptbase64encode_fuzzer/baseptbase64encode_fuzzer.cpp b/test/fuzztest/base/ptbase64/baseptbase64encode_fuzzer/baseptbase64encode_fuzzer.cpp index 91245d81..521d7cf7 100644 --- a/test/fuzztest/base/ptbase64/baseptbase64encode_fuzzer/baseptbase64encode_fuzzer.cpp +++ b/test/fuzztest/base/ptbase64/baseptbase64encode_fuzzer/baseptbase64encode_fuzzer.cpp @@ -33,6 +33,9 @@ namespace OHOS { if (size <= 0) { return; } + if (size > MAXBYTELEN) { + size = MAXBYTELEN; + } auto src = cn->GetString(data, size); std::string dest; dest.resize(PtBase64::EncodedSize(src.size())); diff --git a/test/fuzztest/base/ptevents/breakpointresolved/baseptsbreakpointgetbreakpointid_fuzzer/baseptsbreakpointgetbreakpointid_fuzzer.cpp b/test/fuzztest/base/ptevents/breakpointresolved/baseptsbreakpointgetbreakpointid_fuzzer/baseptsbreakpointgetbreakpointid_fuzzer.cpp index b10ce3ee..e041da6d 100644 --- a/test/fuzztest/base/ptevents/breakpointresolved/baseptsbreakpointgetbreakpointid_fuzzer/baseptsbreakpointgetbreakpointid_fuzzer.cpp +++ b/test/fuzztest/base/ptevents/breakpointresolved/baseptsbreakpointgetbreakpointid_fuzzer/baseptsbreakpointgetbreakpointid_fuzzer.cpp @@ -33,6 +33,9 @@ namespace OHOS { if (size <= 0) { return; } + if (size > MAXBYTELEN) { + size = MAXBYTELEN; + } auto str = cn->GetString(data, size); BreakpointResolved breakpointResolved; breakpointResolved.GetBreakpointId(); diff --git a/test/fuzztest/base/ptevents/breakpointresolved/baseptsbreakpointgetlocation_fuzzer/baseptsbreakpointgetlocation_fuzzer.cpp b/test/fuzztest/base/ptevents/breakpointresolved/baseptsbreakpointgetlocation_fuzzer/baseptsbreakpointgetlocation_fuzzer.cpp index 8e0defb9..bfb74f46 100644 --- a/test/fuzztest/base/ptevents/breakpointresolved/baseptsbreakpointgetlocation_fuzzer/baseptsbreakpointgetlocation_fuzzer.cpp +++ b/test/fuzztest/base/ptevents/breakpointresolved/baseptsbreakpointgetlocation_fuzzer/baseptsbreakpointgetlocation_fuzzer.cpp @@ -33,6 +33,9 @@ namespace OHOS { if (size <= 0) { return; } + if (size > MAXBYTELEN) { + size = MAXBYTELEN; + } auto str = cn->GetString(data, size); BreakpointResolved breakpointResolved; breakpointResolved.GetLocation(); diff --git a/test/fuzztest/base/ptevents/breakpointresolved/baseptsbreakpointgetname_fuzzer/baseptsbreakpointgetname_fuzzer.cpp b/test/fuzztest/base/ptevents/breakpointresolved/baseptsbreakpointgetname_fuzzer/baseptsbreakpointgetname_fuzzer.cpp index 3499d097..604efb8c 100644 --- a/test/fuzztest/base/ptevents/breakpointresolved/baseptsbreakpointgetname_fuzzer/baseptsbreakpointgetname_fuzzer.cpp +++ b/test/fuzztest/base/ptevents/breakpointresolved/baseptsbreakpointgetname_fuzzer/baseptsbreakpointgetname_fuzzer.cpp @@ -33,6 +33,9 @@ namespace OHOS { if (size <= 0) { return; } + if (size > MAXBYTELEN) { + size = MAXBYTELEN; + } auto str = cn->GetString(data, size); auto breakpointResolved = std::make_unique(); breakpointResolved->GetName(); diff --git a/test/fuzztest/base/ptevents/paused/baseptspausedgetcallframes_fuzzer/baseptspausedgetcallframes_fuzzer.cpp b/test/fuzztest/base/ptevents/paused/baseptspausedgetcallframes_fuzzer/baseptspausedgetcallframes_fuzzer.cpp index 3cff0a9f..e7bff0ab 100644 --- a/test/fuzztest/base/ptevents/paused/baseptspausedgetcallframes_fuzzer/baseptspausedgetcallframes_fuzzer.cpp +++ b/test/fuzztest/base/ptevents/paused/baseptspausedgetcallframes_fuzzer/baseptspausedgetcallframes_fuzzer.cpp @@ -33,6 +33,9 @@ namespace OHOS { if (size <= 0) { return; } + if (size > MAXBYTELEN) { + size = MAXBYTELEN; + } auto str = cn->GetString(data, size); Paused paused; paused.GetCallFrames(); diff --git a/test/fuzztest/base/ptevents/paused/baseptspausedgetdata_fuzzer/baseptspausedgetdata_fuzzer.cpp b/test/fuzztest/base/ptevents/paused/baseptspausedgetdata_fuzzer/baseptspausedgetdata_fuzzer.cpp index b596dfa8..db86b818 100644 --- a/test/fuzztest/base/ptevents/paused/baseptspausedgetdata_fuzzer/baseptspausedgetdata_fuzzer.cpp +++ b/test/fuzztest/base/ptevents/paused/baseptspausedgetdata_fuzzer/baseptspausedgetdata_fuzzer.cpp @@ -33,6 +33,9 @@ namespace OHOS { if (size <= 0) { return; } + if (size > MAXBYTELEN) { + size = MAXBYTELEN; + } auto str = cn->GetString(data, size); Paused paused; paused.GetData(); diff --git a/test/fuzztest/base/ptevents/paused/baseptspausedgethitbreakpoints_fuzzer/baseptspausedgethitbreakpoints_fuzzer.cpp b/test/fuzztest/base/ptevents/paused/baseptspausedgethitbreakpoints_fuzzer/baseptspausedgethitbreakpoints_fuzzer.cpp index 865283e5..fbc1ac8c 100644 --- a/test/fuzztest/base/ptevents/paused/baseptspausedgethitbreakpoints_fuzzer/baseptspausedgethitbreakpoints_fuzzer.cpp +++ b/test/fuzztest/base/ptevents/paused/baseptspausedgethitbreakpoints_fuzzer/baseptspausedgethitbreakpoints_fuzzer.cpp @@ -33,6 +33,9 @@ namespace OHOS { if (size <= 0) { return; } + if (size > MAXBYTELEN) { + size = MAXBYTELEN; + } auto str = cn->GetString(data, size); Paused paused; paused.GetHitBreakpoints(); diff --git a/test/fuzztest/base/ptevents/paused/baseptspausedgetname_fuzzer/baseptspausedgetname_fuzzer.cpp b/test/fuzztest/base/ptevents/paused/baseptspausedgetname_fuzzer/baseptspausedgetname_fuzzer.cpp index 0e254368..f9875d4c 100644 --- a/test/fuzztest/base/ptevents/paused/baseptspausedgetname_fuzzer/baseptspausedgetname_fuzzer.cpp +++ b/test/fuzztest/base/ptevents/paused/baseptspausedgetname_fuzzer/baseptspausedgetname_fuzzer.cpp @@ -33,6 +33,9 @@ namespace OHOS { if (size <= 0) { return; } + if (size > MAXBYTELEN) { + size = MAXBYTELEN; + } auto str = cn->GetString(data, size); Paused paused; paused.GetName(); diff --git a/test/fuzztest/base/ptevents/paused/baseptspausedgetreason_fuzzer/baseptspausedgetreason_fuzzer.cpp b/test/fuzztest/base/ptevents/paused/baseptspausedgetreason_fuzzer/baseptspausedgetreason_fuzzer.cpp index 11036971..6eadcfd1 100644 --- a/test/fuzztest/base/ptevents/paused/baseptspausedgetreason_fuzzer/baseptspausedgetreason_fuzzer.cpp +++ b/test/fuzztest/base/ptevents/paused/baseptspausedgetreason_fuzzer/baseptspausedgetreason_fuzzer.cpp @@ -33,6 +33,9 @@ namespace OHOS { if (size <= 0) { return; } + if (size > MAXBYTELEN) { + size = MAXBYTELEN; + } auto str = cn->GetString(data, size); Paused paused; paused.GetReason(); diff --git a/test/fuzztest/protocolhandlerwait_fuzzer/protocolhandlerwait_fuzzer.cpp b/test/fuzztest/protocolhandlerwait_fuzzer/protocolhandlerwait_fuzzer.cpp index 51e930e5..af393de4 100644 --- a/test/fuzztest/protocolhandlerwait_fuzzer/protocolhandlerwait_fuzzer.cpp +++ b/test/fuzztest/protocolhandlerwait_fuzzer/protocolhandlerwait_fuzzer.cpp @@ -32,7 +32,8 @@ namespace OHOS { return; } using ProtoHandler = const std::function; - ProtoHandler ph = [data, size](const void *d, [[maybe_unused]] const std::string &s) -> void { + ProtoHandler ph = [data, size] + ([[maybe_unused]] const void *d, [[maybe_unused]] const std::string &s) -> void { d = data + size; }; ProtocolHandler handler(ph, vm); -- Gitee