diff --git a/test/fuzztest/backupext_fuzzer/backupext_fuzzer.cpp b/test/fuzztest/backupext_fuzzer/backupext_fuzzer.cpp index 158ced24f55d55f798d96a7727d3444eb41f202a..9cb7b8d55edb28275638e5d2eaca7577abc88384 100644 --- a/test/fuzztest/backupext_fuzzer/backupext_fuzzer.cpp +++ b/test/fuzztest/backupext_fuzzer/backupext_fuzzer.cpp @@ -57,6 +57,7 @@ bool OnRemoteRequestFuzzTest(shared_ptr extension, const uin bool InitFuzzTest(shared_ptr backup, const uint8_t *data, size_t size) { + (void)data; shared_ptr record = nullptr; shared_ptr application = nullptr; shared_ptr handler = nullptr; @@ -96,20 +97,22 @@ bool OnConnectFuzzTest(shared_ptr backup, const uint8_t *data, size_t bool CreateFuzzTest(shared_ptr backup, const uint8_t *data, size_t size) { - unique_ptr runtime; + (void)data; + unique_ptr runtime = nullptr; backup->Create(runtime); return true; } bool GetExtensionActionFuzzTest(shared_ptr backup, const uint8_t *data, size_t size) { + (void)data; backup->GetExtensionAction(); return true; } bool OnRestoreFuzzTest(shared_ptr backup, const uint8_t *data, size_t size) { - function callback; + function callback = [data](ErrCode, string) {}; function callbackEx; backup->OnRestore(callback, callbackEx); return true; @@ -117,25 +120,28 @@ bool OnRestoreFuzzTest(shared_ptr backup, const uint8_t *data, size_t bool GetBackupInfoFuzzTest(shared_ptr backup, const uint8_t *data, size_t size) { - function callback = nullptr; + function callback = [data](ErrCode, string) {}; backup->GetBackupInfo(callback); return true; } bool WasFromSpecialVersionFuzzTest(shared_ptr backup, const uint8_t *data, size_t size) { + (void)data; backup->WasFromSpecialVersion(); return true; } bool SpecialVersionForCloneAndCloudFuzzTest(shared_ptr backup, const uint8_t *data, size_t size) { + (void)data; backup->SpecialVersionForCloneAndCloud(); return true; } bool RestoreDataReadyFuzzTest(shared_ptr backup, const uint8_t *data, size_t size) { + (void)data; backup->RestoreDataReady(); return true; } @@ -149,6 +155,7 @@ bool InvokeAppExtMethodFuzzTest(shared_ptr backup, const uint8_t *dat bool SetCreatorFuzzTest(shared_ptr backup, const uint8_t *data, size_t size) { + (void)data; CreatorFunc creator; backup->SetCreator(creator); return true; @@ -169,6 +176,7 @@ bool CmdHandleClearFuzzTest(shared_ptr extension, const uint MessageParcel msg; MessageParcel reply; + msg.WriteBuffer(data, size); extension->CmdHandleClear(msg, reply); return true; } @@ -269,6 +277,7 @@ bool CmdGetIncrementalBackupFileHandleFuzzTest(shared_ptr ex MessageParcel msg; MessageParcel reply; + msg.WriteBuffer(data, size); extension->CmdGetIncrementalBackupFileHandle(msg, reply); return true; } diff --git a/test/fuzztest/backupservicestub_fuzzer/backupservicestub_fuzzer.cpp b/test/fuzztest/backupservicestub_fuzzer/backupservicestub_fuzzer.cpp index b3c714e657930d70deb33921cba61a1f83e6e395..a23175a130750dc45f615dd99ef56889bd3c4433 100644 --- a/test/fuzztest/backupservicestub_fuzzer/backupservicestub_fuzzer.cpp +++ b/test/fuzztest/backupservicestub_fuzzer/backupservicestub_fuzzer.cpp @@ -79,6 +79,7 @@ bool CmdInitRestoreSessionFuzzTest(sptr service, const uint8_t *data, s MessageParcel reply; try { + msg.WriteBuffer(data, size); BSessionRestore::Callbacks callbacks; msg.WriteRemoteObject(new ServiceReverse(callbacks)); service->CmdInitRestoreSession(msg, reply); @@ -94,6 +95,7 @@ bool CmdInitBackupSessionFuzzTest(sptr service, const uint8_t *data, si MessageParcel reply; try { + msg.WriteBuffer(data, size); BSessionBackup::Callbacks callbacks; msg.WriteRemoteObject(new ServiceReverse(callbacks)); service->CmdInitBackupSession(msg, reply); @@ -109,6 +111,7 @@ bool CmdStartFuzzTest(sptr service, const uint8_t *data, size_t size) MessageParcel reply; try { + msg.WriteBuffer(data, size); service->CmdStart(msg, reply); } catch (OHOS::FileManagement::Backup::BError &err) { // Only filter BError errors, Other results are not expected. @@ -122,6 +125,7 @@ bool CmdGetLocalCapabilitiesFuzzTest(sptr service, const uint8_t *data, MessageParcel reply; try { + msg.WriteBuffer(data, size); service->CmdGetLocalCapabilities(msg, reply); } catch (OHOS::FileManagement::Backup::BError &err) { // Only filter BError errors, Other results are not expected. @@ -337,6 +341,7 @@ bool CmdFinishFuzzTest(sptr service, const uint8_t *data, size_t size) MessageParcel msg; MessageParcel reply; try { + msg.WriteBuffer(data, size); service->CmdFinish(msg, reply); } catch (OHOS::FileManagement::Backup::BError &err) { // Only filter BError errors, Other results are not expected. @@ -349,6 +354,7 @@ bool CmdFinishFuzzTest(sptr service, const uint8_t *data, size_t size) MessageParcel msg; MessageParcel reply; try { + msg.WriteBuffer(data, size); service->CmdRelease(msg, reply); } catch (OHOS::FileManagement::Backup::BError &err) { // Only filter BError errors, Other results are not expected. @@ -377,6 +383,7 @@ bool CmdGetAppLocalListAndDoIncrementalBackupFuzzTest(sptr service, con MessageParcel msg; MessageParcel reply; try { + msg.WriteBuffer(data, size); service->CmdGetAppLocalListAndDoIncrementalBackup(msg, reply); } catch (OHOS::FileManagement::Backup::BError &err) { // Only filter BError errors, Other results are not expected. @@ -392,6 +399,7 @@ bool CmdInitIncrementalBackupSessionFuzzTest(sptr service, const uint8_ try { BIncrementalBackupSession::Callbacks callbacks; msg.WriteRemoteObject(new ServiceReverse(callbacks)); + msg.WriteBuffer(data, size); service->CmdInitIncrementalBackupSession(msg, reply); } catch (OHOS::FileManagement::Backup::BError &err) { // Only filter BError errors, Other results are not expected. diff --git a/test/fuzztest/backupservicestubbranch_fuzzer/backupservicestubbranch_fuzzer.cpp b/test/fuzztest/backupservicestubbranch_fuzzer/backupservicestubbranch_fuzzer.cpp index 4d2ba23426f3c59bd2f856b61fa6478a90a44e6d..e5558102895943f89beac1d98d437e63294ea2fc 100644 --- a/test/fuzztest/backupservicestubbranch_fuzzer/backupservicestubbranch_fuzzer.cpp +++ b/test/fuzztest/backupservicestubbranch_fuzzer/backupservicestubbranch_fuzzer.cpp @@ -227,6 +227,7 @@ bool CmdInitRestoreSessionFuzzTest(sptr service, const uint8_t *data, s MessageParcel reply; try { + msg.WriteBuffer(data, size); service->CmdInitRestoreSession(msg, reply); BSessionRestore::Callbacks callbacks; @@ -250,6 +251,7 @@ bool CmdInitBackupSessionFuzzTest(sptr service, const uint8_t *data, si MessageParcel reply; try { + msg.WriteBuffer(data, size); service->CmdInitBackupSession(msg, reply); BSessionBackup::Callbacks callbacks; @@ -273,6 +275,7 @@ bool CmdStartFuzzTest(sptr service, const uint8_t *data, size_t size) MessageParcel reply; try { + msg.WriteBuffer(data, size); ExpectReturn({false}); service->CmdStart(msg, reply); @@ -290,6 +293,7 @@ bool CmdGetLocalCapabilitiesFuzzTest(sptr service, const uint8_t *data, MessageParcel reply; try { + msg.WriteBuffer(data, size); ExpectReturn({false}); service->CmdGetLocalCapabilities(msg, reply); @@ -622,6 +626,7 @@ bool CmdFinishFuzzTest(sptr service, const uint8_t *data, size_t size) MessageParcel msg; MessageParcel reply; try { + msg.WriteBuffer(data, size); ExpectReturn({false}); service->CmdFinish(msg, reply); @@ -638,6 +643,7 @@ bool CmdFinishFuzzTest(sptr service, const uint8_t *data, size_t size) MessageParcel msg; MessageParcel reply; try { + msg.WriteBuffer(data, size); service->CmdRelease(msg, reply); } catch (OHOS::FileManagement::Backup::BError &err) { // Only filter BError errors, Other results are not expected. @@ -680,6 +686,7 @@ bool CmdGetAppLocalListAndDoIncrementalBackupFuzzTest(sptr service, con MessageParcel msg; MessageParcel reply; try { + msg.WriteBuffer(data, size); ExpectReturn({false}); service->CmdGetAppLocalListAndDoIncrementalBackup(msg, reply); @@ -697,6 +704,7 @@ bool CmdInitIncrementalBackupSessionFuzzTest(sptr service, const uint8_ MessageParcel reply; try { + msg.WriteBuffer(data, size); service->CmdInitIncrementalBackupSession(msg, reply); BIncrementalBackupSession::Callbacks callbacks;