]> git.ipfire.org Git - thirdparty/kea.git/commitdiff
[3932] Ignored remove return status
authorFrancis Dupont <fdupont@isc.org>
Wed, 8 Jul 2015 13:45:22 +0000 (15:45 +0200)
committerFrancis Dupont <fdupont@isc.org>
Wed, 8 Jul 2015 13:45:22 +0000 (15:45 +0200)
src/lib/config/command_socket_factory.cc
src/lib/dhcp/tests/iface_mgr_unittest.cc

index bc4448be257776eb1aac88c9bb30882d2870015c..aa3cbe520e86ef75d8b4da6c28761db2976c4807 100644 (file)
@@ -83,7 +83,7 @@ private:
         // Let's remove the old file. We don't care about any possible
         // errors here. The file should not be there if the file was
         // shut down properly.
-        remove(file_name.c_str());
+        static_cast<void>(remove(file_name.c_str()));
 
         // Set this socket to be non-blocking one.
         if (fcntl(fd, F_SETFL, O_NONBLOCK) !=0 ) {
@@ -100,7 +100,7 @@ private:
         if (bind(fd, (struct sockaddr*)&addr, sizeof(addr))) {
             const char* errmsg = strerror(errno);
             ::close(fd);
-            remove(file_name.c_str());
+            static_cast<void>(remove(file_name.c_str()));
             isc_throw(isc::config::SocketError, "Failed to bind socket " << fd
                       << " to " << file_name << ": " << errmsg);
         }
@@ -114,7 +114,7 @@ private:
         if (status < 0) {
             const char* errmsg = strerror(errno);
             ::close(fd);
-            remove(file_name.c_str());
+            static_cast<void>(remove(file_name.c_str()));
             isc_throw(isc::config::SocketError, "Failed to listen on socket fd="
                       << fd << ", filename=" << file_name << ": " << errmsg);
         }
@@ -181,7 +181,7 @@ private:
         // Close should always succeed. We don't care if we're able to delete
         // the socket or not.
         ::close(sockfd_);
-        remove(filename_.c_str());
+        static_cast<void>(remove(filename_.c_str()));
     }
 
     /// @brief UNIX filename representing this socket
index e3771c6f54691ca3103f4906c2fd1ab905374177..ff8bb068bcaccc13d0c5a6b55bb188ca3f4f22ec 100644 (file)
@@ -478,7 +478,7 @@ TEST_F(IfaceMgrTest, dhcp6Sniffer) {
     // Testing socket operation in a portable way is tricky
     // without interface detection implemented
 
-    remove("interfaces.txt");
+    static_cast<void>(remove("interfaces.txt"));
 
     ofstream interfaces("interfaces.txt", ios::ate);
     interfaces << "eth0 fe80::21e:8cff:fe9b:7349";