What the developer wrote
Authored by jpk68
util: fix includes
What changed, and why it matters
This is a routine code cleanup that adjusts which header files are included in a utility source file. It removes one unused include, replaces broad library includes with more specific ones, and reorganizes the include order. There is no indication this fixes a security vulnerability or changes program behavior.
No security action required. Treat as normal maintenance/refactoring commit.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The commit modifies src/common/util.cpp to refine C/C++ header includes: it removes
Changed components
src/common/util.cppInspect captured patch +4 / −3
diff --git a/src/common/util.cpp b/src/common/util.cpp
index 2aaa1a9..12d8075 100644
--- a/src/common/util.cpp
+++ b/src/common/util.cpp
@@ -40,7 +40,6 @@
#include <sys/types.h>
#include <sys/stat.h>
#include <sys/resource.h>
-#include <unistd.h>
#include <dirent.h>
#include <string.h>
#include <ctype.h>
@@ -80,9 +79,11 @@ using namespace epee;
#include <sys/file.h>
#include <sys/stat.h>
#endif
-#include <boost/filesystem.hpp>
+
+#include <boost/filesystem/path.hpp>
+#include <boost/filesystem/operations.hpp>
#include <boost/algorithm/string.hpp>
-#include <boost/asio.hpp>
+#include <boost/asio/ip/address.hpp>
#include <boost/format.hpp>
#include <openssl/evp.h>
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.