From b471b547c8695c349e009db74f661bec92adb613 Mon Sep 17 00:00:00 2001 From: Francis Dupont Date: Wed, 8 Jul 2015 15:56:56 +0200 Subject: [PATCH] [3932] Ignored remove return status --- src/bin/dhcp6/tests/config_parser_unittest.cc | 4 ++-- src/bin/dhcp6/tests/ctrl_dhcp6_srv_unittest.cc | 4 ++-- src/bin/dhcp6/tests/dhcp6_test_utils.cc | 4 ++-- 3 files changed, 6 insertions(+), 6 deletions(-) diff --git a/src/bin/dhcp6/tests/config_parser_unittest.cc b/src/bin/dhcp6/tests/config_parser_unittest.cc index 46f6d91601..5296bc2d2c 100644 --- a/src/bin/dhcp6/tests/config_parser_unittest.cc +++ b/src/bin/dhcp6/tests/config_parser_unittest.cc @@ -115,8 +115,8 @@ public: resetConfiguration(); // ... and delete the hooks library marker files if present - remove(LOAD_MARKER_FILE); - remove(UNLOAD_MARKER_FILE); + static_cast(remove(LOAD_MARKER_FILE)); + static_cast(remove(UNLOAD_MARKER_FILE)); }; // Checks if config_result (result of DHCP server configuration) has diff --git a/src/bin/dhcp6/tests/ctrl_dhcp6_srv_unittest.cc b/src/bin/dhcp6/tests/ctrl_dhcp6_srv_unittest.cc index 31025fd810..86fc34c437 100644 --- a/src/bin/dhcp6/tests/ctrl_dhcp6_srv_unittest.cc +++ b/src/bin/dhcp6/tests/ctrl_dhcp6_srv_unittest.cc @@ -247,7 +247,7 @@ public: }; void createUnixChannelServer() { - ::remove(socket_path_.c_str()); + static_cast(::remove(socket_path_.c_str())); // Just a simple config. The important part here is the socket // location information. @@ -292,7 +292,7 @@ public: /// @brief Reset void reset() { CtrlDhcpv6SrvTest::reset(); - ::remove(socket_path_.c_str()); + static_cast(::remove(socket_path_.c_str())); } /// @brief Conducts a command/response exchange via UnixCommandSocket diff --git a/src/bin/dhcp6/tests/dhcp6_test_utils.cc b/src/bin/dhcp6/tests/dhcp6_test_utils.cc index 9e6fe0fd99..dce8003834 100644 --- a/src/bin/dhcp6/tests/dhcp6_test_utils.cc +++ b/src/bin/dhcp6/tests/dhcp6_test_utils.cc @@ -834,7 +834,7 @@ Dhcpv6SrvTest::configure(const std::string& config, NakedDhcpv6Srv& srv) { NakedDhcpv6SrvTest::NakedDhcpv6SrvTest() : rcode_(-1) { // it's ok if that fails. There should not be such a file anyway - remove(DUID_FILE); + static_cast(remove(DUID_FILE)); const isc::dhcp::IfaceMgr::IfaceCollection& ifaces = isc::dhcp::IfaceMgr::instance().getIfaces(); @@ -856,7 +856,7 @@ NakedDhcpv6SrvTest::~NakedDhcpv6SrvTest() { isc::stats::StatsMgr::instance().removeAll(); // Let's clean up if there is such a file. - remove(DUID_FILE); + static_cast(remove(DUID_FILE)); isc::hooks::HooksManager::preCalloutsLibraryHandle() .deregisterAllCallouts("buffer6_receive"); isc::hooks::HooksManager::preCalloutsLibraryHandle() -- 2.47.2