Skip to content

Commit 40d21d7

Browse files
authored
Merge pull request containerd#71 from Tim-Zhang/fix-client-fd-leak
Log a warning when close recver_fd failed in client.
2 parents f3471b6 + e5f1430 commit 40d21d7

File tree

1 file changed

+6
-1
lines changed

1 file changed

+6
-1
lines changed

src/sync/client.rs

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -186,7 +186,12 @@ impl Client {
186186
map.remove(&mh.stream_id);
187187
}
188188

189-
close(recver_fd).unwrap();
189+
let _ = close(recver_fd).map_err(|e| {
190+
warn!(
191+
"failed to close recver_fd: {} with error: {:?}",
192+
recver_fd, e
193+
)
194+
});
190195

191196
trace!("Recver quit");
192197
});

0 commit comments

Comments
 (0)