From 61e43cffb98b33874a5594abc35e74dd495ddf36 Mon Sep 17 00:00:00 2001 From: Abtin Keshavarzian Date: Wed, 12 Nov 2025 11:37:17 -0800 Subject: [PATCH] [fuzz] fix warning when building fuzzer source files (#12130) Add `SuccessOrQuit()` to check the return `Error` value of `RoutingManager::SetEnabled()` in `fuzz_*.cpp` source files. This addresses warnings about ignoring the return value. --- tests/fuzz/fuzz_cli.cpp | 2 +- tests/fuzz/fuzz_icmp6.cpp | 2 +- tests/fuzz/fuzz_ip6.cpp | 2 +- tests/fuzz/fuzz_mdns.cpp | 2 +- tests/fuzz/fuzz_radio-one-node.cpp | 2 +- tests/fuzz/fuzz_trel.cpp | 2 +- 6 files changed, 6 insertions(+), 6 deletions(-) diff --git a/tests/fuzz/fuzz_cli.cpp b/tests/fuzz/fuzz_cli.cpp index 2c73e7ab3..42c37c3f1 100644 --- a/tests/fuzz/fuzz_cli.cpp +++ b/tests/fuzz/fuzz_cli.cpp @@ -112,7 +112,7 @@ extern "C" int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) otCliInit(&node.GetInstance(), CliOutput, nullptr); node.GetInstance().Get().Init(/* aInfraIfIndex */ 1, /* aInfraIfIsRunning */ true); - node.GetInstance().Get().SetEnabled(true); + SuccessOrQuit(node.GetInstance().Get().SetEnabled(true)); node.GetInstance().Get().SetAutoEnableMode(true); node.GetInstance().Get().SetDhcp6PdEnabled(true); node.GetInstance().Get().SetNat64PrefixManagerEnabled(true); diff --git a/tests/fuzz/fuzz_icmp6.cpp b/tests/fuzz/fuzz_icmp6.cpp index faf6c6acc..ec5d8cff4 100644 --- a/tests/fuzz/fuzz_icmp6.cpp +++ b/tests/fuzz/fuzz_icmp6.cpp @@ -101,7 +101,7 @@ extern "C" int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) node.GetInstance().SetLogLevel(kLogLevelInfo); node.GetInstance().Get().Init(/* aInfraIfIndex */ 1, /* aInfraIfIsRunning */ true); - node.GetInstance().Get().SetEnabled(true); + SuccessOrQuit(node.GetInstance().Get().SetEnabled(true)); node.GetInstance().Get().SetAutoEnableMode(true); node.GetInstance().Get().SetDhcp6PdEnabled(true); node.GetInstance().Get().SetNat64PrefixManagerEnabled(true); diff --git a/tests/fuzz/fuzz_ip6.cpp b/tests/fuzz/fuzz_ip6.cpp index 56eee3705..46c48e0f6 100644 --- a/tests/fuzz/fuzz_ip6.cpp +++ b/tests/fuzz/fuzz_ip6.cpp @@ -113,7 +113,7 @@ extern "C" int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) node.GetInstance().SetLogLevel(kLogLevelInfo); node.GetInstance().Get().Init(/* aInfraIfIndex */ 1, /* aInfraIfIsRunning */ true); - node.GetInstance().Get().SetEnabled(true); + SuccessOrQuit(node.GetInstance().Get().SetEnabled(true)); node.GetInstance().Get().SetAutoEnableMode(true); node.GetInstance().Get().SetDhcp6PdEnabled(true); node.GetInstance().Get().SetNat64PrefixManagerEnabled(true); diff --git a/tests/fuzz/fuzz_mdns.cpp b/tests/fuzz/fuzz_mdns.cpp index 940983ff4..b4b13bbb4 100644 --- a/tests/fuzz/fuzz_mdns.cpp +++ b/tests/fuzz/fuzz_mdns.cpp @@ -112,7 +112,7 @@ extern "C" int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) node.GetInstance().SetLogLevel(kLogLevelInfo); node.GetInstance().Get().Init(/* aInfraIfIndex */ 1, /* aInfraIfIsRunning */ true); - node.GetInstance().Get().SetEnabled(true); + SuccessOrQuit(node.GetInstance().Get().SetEnabled(true)); node.GetInstance().Get().SetAutoEnableMode(true); node.GetInstance().Get().SetDhcp6PdEnabled(true); node.GetInstance().Get().SetNat64PrefixManagerEnabled(true); diff --git a/tests/fuzz/fuzz_radio-one-node.cpp b/tests/fuzz/fuzz_radio-one-node.cpp index aa4e229c2..1c799488e 100644 --- a/tests/fuzz/fuzz_radio-one-node.cpp +++ b/tests/fuzz/fuzz_radio-one-node.cpp @@ -111,7 +111,7 @@ extern "C" int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) node.GetInstance().SetLogLevel(kLogLevelInfo); node.GetInstance().Get().Init(/* aInfraIfIndex */ 1, /* aInfraIfIsRunning */ true); - node.GetInstance().Get().SetEnabled(true); + SuccessOrQuit(node.GetInstance().Get().SetEnabled(true)); node.GetInstance().Get().SetAutoEnableMode(true); node.GetInstance().Get().SetDhcp6PdEnabled(true); node.GetInstance().Get().SetNat64PrefixManagerEnabled(true); diff --git a/tests/fuzz/fuzz_trel.cpp b/tests/fuzz/fuzz_trel.cpp index 997598df3..8bdb63d24 100644 --- a/tests/fuzz/fuzz_trel.cpp +++ b/tests/fuzz/fuzz_trel.cpp @@ -102,7 +102,7 @@ extern "C" int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) node.GetInstance().SetLogLevel(kLogLevelInfo); node.GetInstance().Get().Init(/* aInfraIfIndex */ 1, /* aInfraIfIsRunning */ true); - node.GetInstance().Get().SetEnabled(true); + SuccessOrQuit(node.GetInstance().Get().SetEnabled(true)); node.GetInstance().Get().SetAutoEnableMode(true); node.GetInstance().Get().SetDhcp6PdEnabled(true); node.GetInstance().Get().SetNat64PrefixManagerEnabled(true);