Move fd vector clearing into update-function

This way is more self-contained and leaves less room
for accidentall fd reuse.

2 files changed, 1 insertions, 1 deletions

diff --git a/configd/src/apps/sentinel/manager.cpp b/configd/src/apps/sentinel/manager.cpp
index f6a74774610..9ee259bb892 100644
--- a/configd/src/apps/sentinel/manager.cpp
+++ b/configd/src/apps/sentinel/manager.cpp
@@ -172,6 +172,7 @@ Manager::handleChildDeaths()
 void
 Manager::updateActiveFdset(std::vector<pollfd> &fds)
 {
+    fds.clear();
     for (const OutputConnection *c : _outputConnections) {
         int fd = c->fd();
         if (fd >= 0) {
diff --git a/configd/src/apps/sentinel/sentinel.cpp b/configd/src/apps/sentinel/sentinel.cpp
index a327769e706..db9f73ea76d 100644
--- a/configd/src/apps/sentinel/sentinel.cpp
+++ b/configd/src/apps/sentinel/sentinel.cpp
@@ -103,7 +103,6 @@ main(int argc, char **argv)
         if (vespalib::SignalHandler::CHLD.check()) {
             continue;
         }
-        fds.clear();
         manager.updateActiveFdset(fds);
         constexpr int poll_timeout_ms = 100;