From b38841ca2df7a70dd7464f6ded3fb141270930a9 Mon Sep 17 00:00:00 2001 From: Nicholas Marriott Date: Thu, 19 Aug 2010 17:20:27 +0000 Subject: Do not need to dup() the tty fd sent from the client because it is already dup()d again later. Fixes a leak seen by espie@. --- usr.bin/tmux/server-client.c | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-) (limited to 'usr.bin/tmux') diff --git a/usr.bin/tmux/server-client.c b/usr.bin/tmux/server-client.c index 3bc813ce45f..ce952df4103 100644 --- a/usr.bin/tmux/server-client.c +++ b/usr.bin/tmux/server-client.c @@ -1,4 +1,4 @@ -/* $OpenBSD: server-client.c,v 1.38 2010/08/11 07:34:43 nicm Exp $ */ +/* $OpenBSD: server-client.c,v 1.39 2010/08/19 17:20:26 nicm Exp $ */ /* * Copyright (c) 2009 Nicholas Marriott @@ -696,17 +696,15 @@ server_client_msg_dispatch(struct client *c) fatalx("MSG_IDENTIFY missing fd"); memcpy(&identifydata, imsg.data, sizeof identifydata); - c->stdin_fd = dup(imsg.fd); - if (c->stdin_fd == -1) - fatal("dup failed"); + c->stdin_fd = imsg.fd; c->stdin_event = bufferevent_new(c->stdin_fd, NULL, NULL, server_client_in_callback, c); if (c->stdin_event == NULL) fatalx("failed to create stdin event"); - if ((mode = fcntl(imsg.fd, F_GETFL)) != -1) - fcntl(imsg.fd, F_SETFL, mode|O_NONBLOCK); - if (fcntl(imsg.fd, F_SETFD, FD_CLOEXEC) == -1) + if ((mode = fcntl(c->stdin_fd, F_GETFL)) != -1) + fcntl(c->stdin_fd, F_SETFL, mode|O_NONBLOCK); + if (fcntl(c->stdin_fd, F_SETFD, FD_CLOEXEC) == -1) fatal("fcntl failed"); server_client_msg_identify(c, &identifydata, imsg.fd); -- cgit v1.2.3