commons: close fds we dup-ed after sending them

CHANGELOG.md

@@ -29,6 +29,8 @@
   design change above.
 - [scanner] Fixed a number of cases where invalid Rust code would be generated if variable
   or method names in the protocol were Rust keywords.
+- [commons] Properly close FDs we dup-ed after sending them to the server, to avoid leaking
+  open FDs.
 
 ## 0.21.11 -- 2019-01-19
 

wayland-commons/src/socket.rs

@@ -143,6 +143,10 @@ impl BufferedSocket {
             let bytes = unsafe { ::std::slice::from_raw_parts(words.as_ptr() as *const u8, words.len() * 4) };
             let fds = self.out_fds.get_contents();
             self.socket.send_msg(bytes, fds)?;
+            for &fd in fds {
+                // once the fds are sent, we can close them
+                let _ = ::nix::unistd::close(fd);
+            }
         }
         self.out_data.clear();
         self.out_fds.clear();