[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 08/16] qemu-char: Close fd at end of file
From: |
minyard |
Subject: |
[Qemu-devel] [PATCH 08/16] qemu-char: Close fd at end of file |
Date: |
Sun, 15 Jul 2012 15:25:03 -0500 |
From: Corey Minyard <address@hidden>
The chardev backends that used qemu_chr_open_fd did not get their
file descriptors closed at end of file or when the chardev was closed.
This could result in a file descriptor leak.
Signed-off-by: Corey Minyard <address@hidden>
---
qemu-char.c | 29 +++++++++++++++++++++++------
1 files changed, 23 insertions(+), 6 deletions(-)
diff --git a/qemu-char.c b/qemu-char.c
index 8fbc704..9cbc8c0 100644
--- a/qemu-char.c
+++ b/qemu-char.c
@@ -563,6 +563,7 @@ static int stdio_nb_clients;
typedef struct {
int fd_in, fd_out;
int max_size;
+ int close_fds_on_close;
} FDCharDriver;
@@ -581,6 +582,18 @@ static int fd_chr_read_poll(void *opaque)
return s->max_size;
}
+static void fd_close_fds(FDCharDriver *s)
+{
+ if (!s->close_fds_on_close)
+ return;
+
+ if ((s->fd_out != s->fd_in) && (s->fd_out != -1))
+ close(s->fd_out);
+ s->fd_out = -1;
+ close(s->fd_in);
+ s->fd_in = -1;
+}
+
static void fd_chr_read(void *opaque)
{
CharDriverState *chr = opaque;
@@ -597,6 +610,7 @@ static void fd_chr_read(void *opaque)
if (size == 0) {
/* FD has been closed. Remove it from the active list. */
qemu_set_fd_handler2(s->fd_in, NULL, NULL, NULL, NULL);
+ fd_close_fds(s);
qemu_chr_be_event(chr, CHR_EVENT_CLOSED);
return;
}
@@ -626,6 +640,7 @@ static void fd_chr_close(struct CharDriverState *chr)
if (display_type == DT_NOGRAPHIC && s->fd_in == 0) {
} else {
qemu_set_fd_handler2(s->fd_in, NULL, NULL, NULL, NULL);
+ fd_close_fds(s);
}
}
@@ -635,13 +650,15 @@ static void fd_chr_close(struct CharDriverState *chr)
/* open a character device to a unix fd */
static CharDriverState *qemu_chr_open_fd(CharDriverState *chr,
- int fd_in, int fd_out)
+ int fd_in, int fd_out,
+ int close_fds_on_close)
{
FDCharDriver *s;
s = g_malloc0(sizeof(FDCharDriver));
s->fd_in = fd_in;
s->fd_out = fd_out;
+ s->close_fds_on_close = close_fds_on_close;
chr->opaque = s;
chr->chr_write = fd_chr_write;
chr->chr_update_read_handler = fd_chr_update_read_handler;
@@ -662,7 +679,7 @@ static CharDriverState
*qemu_chr_open_file_out(CharDriverState *chr,
if (fd_out < 0) {
return NULL;
}
- return qemu_chr_open_fd(chr, -1, fd_out);
+ return qemu_chr_open_fd(chr, -1, fd_out, 1);
}
static CharDriverState *qemu_chr_open_pipe(CharDriverState *chr, QemuOpts
*opts)
@@ -690,7 +707,7 @@ static CharDriverState *qemu_chr_open_pipe(CharDriverState
*chr, QemuOpts *opts)
return NULL;
}
}
- return qemu_chr_open_fd(chr, fd_in, fd_out);
+ return qemu_chr_open_fd(chr, fd_in, fd_out, 1);
}
@@ -794,7 +811,7 @@ static CharDriverState *qemu_chr_open_stdio(CharDriverState
*chr,
atexit(term_exit);
}
- qemu_chr_open_fd(chr, 0, 1);
+ qemu_chr_open_fd(chr, 0, 1, 0);
chr->chr_close = qemu_chr_close_stdio;
chr->chr_set_echo = qemu_chr_set_echo_stdio;
qemu_set_fd_handler2(0, stdio_read_poll, stdio_read, NULL, chr);
@@ -1239,7 +1256,7 @@ static CharDriverState *qemu_chr_open_tty(CharDriverState
*chr, QemuOpts *opts)
return NULL;
}
tty_serial_init(fd, 115200, 'N', 8, 1);
- qemu_chr_open_fd(chr, fd, fd);
+ qemu_chr_open_fd(chr, fd, fd, 1);
chr->chr_ioctl = tty_serial_ioctl;
chr->chr_close = qemu_chr_close_tty;
return chr;
@@ -2313,7 +2330,7 @@ static void tcp_chr_read(void *opaque)
CharDriverState *qemu_chr_open_eventfd(int eventfd)
{
CharDriverState *chr = g_malloc0(sizeof(CharDriverState));
- return qemu_chr_open_fd(chr, eventfd, eventfd);
+ return qemu_chr_open_fd(chr, eventfd, eventfd, 0);
}
#endif
--
1.7.4.1
- [Qemu-devel] [PATCH 15/16] IPMI: Add a local BMC simulation, (continued)
- [Qemu-devel] [PATCH 15/16] IPMI: Add a local BMC simulation, minyard, 2012/07/15
- [Qemu-devel] [PATCH 02/16] pc: move SMBIOS setup to after device init, minyard, 2012/07/15
- [Qemu-devel] [PATCH 03/16] vl: Move init_timer_alarm() earlier, minyard, 2012/07/15
- [Qemu-devel] [PATCH 14/16] IPMI: Add a BT low-level interface, minyard, 2012/07/15
- [Qemu-devel] [PATCH 16/16] IPMI: Add an external connection simulation interface, minyard, 2012/07/15
- [Qemu-devel] [PATCH 05/16] qemu-char: Allow a chardev to reconnect if disconnected, minyard, 2012/07/15
- [Qemu-devel] [PATCH 06/16] qemu-char: Fix a race reporting opens and closes, minyard, 2012/07/15
- [Qemu-devel] [PATCH 04/16] qemu-char: Allocate CharDriverState in qemu_chr_new_from_opts, minyard, 2012/07/15
- [Qemu-devel] [PATCH 11/16] Add a base IPMI interface, minyard, 2012/07/15
- [Qemu-devel] [PATCH 09/16] qdev: Add a pre-firmware init capability, minyard, 2012/07/15
- [Qemu-devel] [PATCH 08/16] qemu-char: Close fd at end of file,
minyard <=
- [Qemu-devel] [PATCH 12/16] IPMI: Add a PC ISA type structure, minyard, 2012/07/15
- [Qemu-devel] [PATCH 13/16] IPMI: Add a KCS low-level interface, minyard, 2012/07/15
- Re: [Qemu-devel] [PATCH 01/16] smbios: Add a function to directly add an entry, Corey Minyard, 2012/07/15
- Re: [Qemu-devel] [PATCH 01/16] smbios: Add a function to directly add an entry, Eric Blake, 2012/07/16