ScopedFD - only use move semantics

justsmth · Nov 13, 2024 · d70832d · d70832d
1 parent 1836a5f
commit d70832d
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/crypto/test/file_util.h b/crypto/test/file_util.h
@@ -46,12 +46,15 @@ class ScopedFD {
   explicit ScopedFD(int fd) : fd_(fd) {}
   ~ScopedFD() { reset(); }
 
-  ScopedFD(ScopedFD &&other) { *this = std::move(other); }
-  ScopedFD &operator=(ScopedFD other) {
+  ScopedFD(ScopedFD &&other) noexcept { *this = std::move(other); }
+  ScopedFD &operator=(ScopedFD&& other) {
     reset(other.release());
     return *this;
   }
 
+  ScopedFD(const ScopedFD &other) = delete;
+  ScopedFD &operator=(ScopedFD& other) = delete;
+
   bool is_valid() const { return fd_ >= 0; }
   int get() const { return fd_; }