use unsafe close

replit · Jun 6, 2024 · 23b6009 · 23b6009
1 parent 42f3552
commit 23b6009
Showing 1 changed file with 16 additions and 2 deletions.
diff --git a/src/lib.rs b/src/lib.rs
@@ -303,11 +303,25 @@ impl Pty {
   #[allow(dead_code)]
   pub fn close(&mut self) -> Result<(), napi::Error> {
     if let Some(fd) = self.controller_fd.take() {
-      drop(fd);
+      unsafe {
+        if libc::close(fd.as_raw_fd()) == -1 {
+          return Err(napi::Error::new(
+            napi::Status::GenericFailure,
+            format!("close failed: {}", Error::last_os_error()),
+          ));
+        }
+      };
     }
 
     if let Some(fd) = self.user_fd.take() {
-      drop(fd);
+      unsafe {
+        if libc::close(fd.as_raw_fd()) == -1 {
+          return Err(napi::Error::new(
+            napi::Status::GenericFailure,
+            format!("close failed: {}", Error::last_os_error()),
+          ));
+        }
+      };
     }
 
     Ok(())