diff --git a/services/loopevent/loop/le_epoll.c b/services/loopevent/loop/le_epoll.c index a47134de960250cda205eb28c17c928260ca3c77..947695e3a75f4fef6766046dd86f4024f2d97366 100644 --- a/services/loopevent/loop/le_epoll.c +++ b/services/loopevent/loop/le_epoll.c @@ -16,7 +16,6 @@ #include #include "le_epoll.h" -#include "le_utils.h" static int IsValid_(const EventEpoll *loop) { @@ -139,4 +138,4 @@ LE_STATUS CreateEpollLoop(EventLoop **loop, uint32_t maxevents, uint32_t timeout epoll->loop.addEvent = AddEvent_; epoll->loop.modEvent = ModEvent_; return LE_SUCCESS; -} \ No newline at end of file +} diff --git a/services/modules/init_eng/init_eng.c b/services/modules/init_eng/init_eng.c index 9b365963af874df781bae2a0963ee06eef624978..921c43d36bb53e1ccb10788268f9d0aeaa3fc414 100644 --- a/services/modules/init_eng/init_eng.c +++ b/services/modules/init_eng/init_eng.c @@ -72,7 +72,6 @@ ENG_STATIC void BuildMountCmd(char *buffer, size_t len, const char *mp, const ch { int ret = snprintf_s(buffer, len, len - 1, "%s %s %s ro barrier=1", fstype, dev, mp); - if (ret == -1) { *buffer = '\0'; } diff --git a/services/utils/init_utils.c b/services/utils/init_utils.c index d1e545915631fd7d75d67ad239d78d42892c024c..7d05fda07c0b73aaee529d0397855c195fbefeff 100644 --- a/services/utils/init_utils.c +++ b/services/utils/init_utils.c @@ -206,6 +206,7 @@ int IterateNameValuePairs(const char *src, void (*iterator)(const NAME_VALUE_PAI { int cnt = 0; const char *seperator; + const char *tmp = src; NAME_VALUE_PAIR nv; if ((src == NULL) || (iterator == NULL)) { return -1; @@ -213,15 +214,15 @@ int IterateNameValuePairs(const char *src, void (*iterator)(const NAME_VALUE_PAI do { // Find space seperator - nv.name = src; - seperator = strchr(src, ' '); + nv.name = tmp; + seperator = strchr(tmp, ' '); if (seperator == NULL) { // Last nv - nv.valueEnd = src + strlen(src); - src = NULL; + nv.valueEnd = tmp + strlen(tmp); + tmp = NULL; } else { nv.valueEnd = seperator; - src = seperator + 1; + tmp = seperator + 1; } // Find equal seperator @@ -239,7 +240,7 @@ int IterateNameValuePairs(const char *src, void (*iterator)(const NAME_VALUE_PAI iterator(&nv, context); cnt += 1; - } while (src != NULL); + } while (tmp != NULL); return cnt; } diff --git a/test/fuzztest/DoFormat_fuzzer/DoFormat_fuzzer.cpp b/test/fuzztest/DoFormat_fuzzer/DoFormat_fuzzer.cpp index e52f02de64144911de910bab84d30d9935ae52c2..745b7d4b3736049bbc5158323bc6524fc80b3b51 100644 --- a/test/fuzztest/DoFormat_fuzzer/DoFormat_fuzzer.cpp +++ b/test/fuzztest/DoFormat_fuzzer/DoFormat_fuzzer.cpp @@ -13,8 +13,6 @@ * limitations under the License. */ -#include "DoFormat_fuzzer.h" -#include #include #include "fs_manager/fs_manager.h" diff --git a/test/fuzztest/GetMountStatusForMountPoint_fuzzer/GetMountStatusForMountPoint_fuzzer.cpp b/test/fuzztest/GetMountStatusForMountPoint_fuzzer/GetMountStatusForMountPoint_fuzzer.cpp index 93d783c1399ea0679773fb925c42596a8c3346fb..e627188f436788e0e98cc592def6d272a2cb967b 100644 --- a/test/fuzztest/GetMountStatusForMountPoint_fuzzer/GetMountStatusForMountPoint_fuzzer.cpp +++ b/test/fuzztest/GetMountStatusForMountPoint_fuzzer/GetMountStatusForMountPoint_fuzzer.cpp @@ -14,7 +14,6 @@ */ #include "GetMountStatusForMountPoint_fuzzer.h" -#include #include #include "fs_manager/fs_manager.h" diff --git a/test/fuzztest/IsSupportedFilesystem_fuzzer/IsSupportedFilesystem_fuzzer.cpp b/test/fuzztest/IsSupportedFilesystem_fuzzer/IsSupportedFilesystem_fuzzer.cpp index 07ddd54f4a2cde1a093a640168ce2fea3ace5706..900d944d67083c50954e52d92975871743498218 100644 --- a/test/fuzztest/IsSupportedFilesystem_fuzzer/IsSupportedFilesystem_fuzzer.cpp +++ b/test/fuzztest/IsSupportedFilesystem_fuzzer/IsSupportedFilesystem_fuzzer.cpp @@ -14,7 +14,6 @@ */ #include "IsSupportedFilesystem_fuzzer.h" -#include #include #include "fs_manager/fs_manager.h" diff --git a/test/fuzztest/LoadFscryptPolicy_fuzzer/LoadFscryptPolicy_fuzzer.cpp b/test/fuzztest/LoadFscryptPolicy_fuzzer/LoadFscryptPolicy_fuzzer.cpp index 50d2f10cb57574581336d07749bfc33674de9ffe..c5a5334a5890da45f6457b96590f5d58fbea1078 100644 --- a/test/fuzztest/LoadFscryptPolicy_fuzzer/LoadFscryptPolicy_fuzzer.cpp +++ b/test/fuzztest/LoadFscryptPolicy_fuzzer/LoadFscryptPolicy_fuzzer.cpp @@ -14,7 +14,6 @@ */ #include "LoadFscryptPolicy_fuzzer.h" -#include #include "fs_manager/fs_manager.h" namespace OHOS { diff --git a/test/unittest/deviceinfo/DeviceInfoUnittest.cpp b/test/unittest/deviceinfo/DeviceInfoUnittest.cpp index 114c232a4ab0475789155f3d0b88b1e2e2481374..7fe745df64e10627fa6aef21051b450e8a950739 100644 --- a/test/unittest/deviceinfo/DeviceInfoUnittest.cpp +++ b/test/unittest/deviceinfo/DeviceInfoUnittest.cpp @@ -36,7 +36,7 @@ using namespace testing::ext; using namespace std; using namespace OHOS; -int g_tokenType = OHOS::Security::AccessToken::TOKEN_HAP; +static int g_tokenType = OHOS::Security::AccessToken::TOKEN_HAP; int g_tokenVerifyResult = 0; namespace OHOS { namespace Security { diff --git a/test/unittest/loopevent/loopserver_unittest.cpp b/test/unittest/loopevent/loopserver_unittest.cpp index d44779d5f8071c5fdaaec0bc5a5b98c7a757f4b5..d43d01738de0d79d201e9e9481d5d7b1350fb504 100644 --- a/test/unittest/loopevent/loopserver_unittest.cpp +++ b/test/unittest/loopevent/loopserver_unittest.cpp @@ -16,6 +16,7 @@ #include #include #include +#include #include "begetctl.h" #include "cJSON.h" @@ -62,7 +63,8 @@ static void DecodeMessage(const char *buffer, size_t nread, uint32_t &cmd) return; } -static void SendMessage(const LoopHandle loopHandle, const TaskHandle taskHandle, const char *message, ...) +template +static void SendMessage(const LoopHandle loopHandle, const TaskHandle taskHandle, const char *message, Args... args) { uint32_t bufferSize = 1024; // 1024 buffer size BufferHandle handle = LE_CreateBuffer(loopHandle, bufferSize); @@ -96,21 +98,21 @@ static void TestOnReceiveRequest(const TaskHandle task, const uint8_t *buffer, u if (buffer == nullptr) { return; } - printf("Server receive message %s \n", reinterpret_cast(buffer)); + printf("Server receive message %s \n", static_cast(buffer)); uint32_t cmd = 0; - DecodeMessage(reinterpret_cast(buffer), nread, cmd); - SendMessage(g_loopServer_, task, reinterpret_cast(buffer)); + DecodeMessage(static_cast(buffer), nread, cmd); + SendMessage(g_loopServer_, task, static_cast(buffer)); } static void TestClientOnReceiveRequest(const TaskHandle task, const uint8_t *buffer, uint32_t nread) { - printf("Client receive message %s \n", reinterpret_cast(buffer)); + printf("Client receive message %s \n", static_cast(buffer)); EXPECT_NE(buffer, nullptr); if (buffer == nullptr) { return; } uint32_t cmd = 0; - DecodeMessage(reinterpret_cast(buffer), nread, cmd); + DecodeMessage(static_cast(buffer), nread, cmd); if (cmd == 5 || cmd == 2) { // 2 5 close server LE_StopLoop(g_loopClient_); } @@ -130,7 +132,7 @@ static void TestSendMessageComplete(const TaskHandle taskHandle, BufferHandle ha uint32_t bufferSize = 1024; // 1024 buffer size char *buffer = (char *)LE_GetBufferInfo(handle, nullptr, &bufferSize); uint32_t cmd = 0; - DecodeMessage(reinterpret_cast(buffer), bufferSize, cmd); + DecodeMessage(static_cast(buffer), bufferSize, cmd); if (cmd == 5) { // 5 close server LE_StopLoop(g_loopServer_); } diff --git a/test/unittest/param/param_shell_unittest.cpp b/test/unittest/param/param_shell_unittest.cpp index bf0af31ca89baab94ef1725d86898ff6cf82afee..35afb7ee08d9c0dfae4c16941542de418e889941 100644 --- a/test/unittest/param/param_shell_unittest.cpp +++ b/test/unittest/param/param_shell_unittest.cpp @@ -213,6 +213,5 @@ HWTEST_F(ParamShellUnitTest, TestParamShellcmd2, TestSize.Level1) BShellEnvLoop(nullptr); BShellEnvErrString(GetShellHandle(), 1); BShellEnvOutputResult(GetShellHandle(), 1); - } } // namespace init_ut diff --git a/test/unittest/param/param_stub.h b/test/unittest/param/param_stub.h index 59dd3eaca44bf64bbd9be9a830d10f6cc528b40b..8c7817caccd62a37c2376c48bedb370b10dba3cb 100644 --- a/test/unittest/param/param_stub.h +++ b/test/unittest/param/param_stub.h @@ -76,4 +76,4 @@ ParamLabelIndex *TestGetParamLabelIndex(const char *name); } #endif #endif -#endif // PARAM_TEST_STUB_ \ No newline at end of file +#endif // PARAM_TEST_STUB_