apiserver: fix file descriptor leak in exec

The ChildHandles struct was not properly closing the file descriptor
(pty_fd) when dropped. This caused a leak of one RawFd per apiclient
exec command. Added a Drop implementation to ensure proper cleanup of
the raw file descriptor.

Signed-off-by: Kyle Sessions <[email protected]>

Author: KCSesh

sources/api/apiserver/src/server/exec/child.rs

@@ -241,6 +241,13 @@ impl ChildHandles {
     }
 }
 
+impl Drop for ChildHandles {
+    /// Ensures the RawFd is properly closed when ChildHandles is dropped.
+    fn drop(&mut self) {
+        let _ = close(self.pty_fd);
+    }
+}
+
 /// ChildFds sets up read and write file descriptors for a Command (before it's spawned) based on
 /// whether the user requested a TTY.
 struct ChildFds {