[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PULL 48/50] char: use common error path in qmp_chardev_add
From: |
Paolo Bonzini |
Subject: |
[Qemu-devel] [PULL 48/50] char: use common error path in qmp_chardev_add |
Date: |
Mon, 24 Oct 2016 15:47:33 +0200 |
From: Marc-André Lureau <address@hidden>
Signed-off-by: Marc-André Lureau <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>
---
qemu-char.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/qemu-char.c b/qemu-char.c
index 02adea4..5e9684d 100644
--- a/qemu-char.c
+++ b/qemu-char.c
@@ -4752,8 +4752,7 @@ ChardevReturn *qmp_chardev_add(const char *id,
ChardevBackend *backend,
chr = qemu_chr_find(id);
if (chr) {
error_setg(errp, "Chardev '%s' already exists", id);
- g_free(ret);
- return NULL;
+ goto out_error;
}
for (i = backends; i; i = i->next) {
--
1.8.3.1
- [Qemu-devel] [PULL 43/50] char: move fe_open in CharBackend, (continued)
- [Qemu-devel] [PULL 43/50] char: move fe_open in CharBackend, Paolo Bonzini, 2016/10/24
- [Qemu-devel] [PULL 39/50] tests: start chardev unit tests, Paolo Bonzini, 2016/10/24
- [Qemu-devel] [PULL 45/50] char: use an enum for CHR_EVENT, Paolo Bonzini, 2016/10/24
- [Qemu-devel] [PULL 38/50] char: make some qemu_chr_fe skip if no driver, Paolo Bonzini, 2016/10/24
- [Qemu-devel] [PULL 40/50] char: move front end handlers in CharBackend, Paolo Bonzini, 2016/10/24
- [Qemu-devel] [PULL 46/50] char: remove unused qemu_chr_fe_event, Paolo Bonzini, 2016/10/24
- [Qemu-devel] [PULL 42/50] char: remove explicit_fe_open, use a set_handlers argument, Paolo Bonzini, 2016/10/24
- [Qemu-devel] [PULL 34/50] char: use qemu_chr_fe* functions with CharBackend argument, Paolo Bonzini, 2016/10/24
- [Qemu-devel] [PULL 44/50] char: remove unused CHR_EVENT_FOCUS, Paolo Bonzini, 2016/10/24
- [Qemu-devel] [PULL 47/50] char: replace avail_connections, Paolo Bonzini, 2016/10/24
- [Qemu-devel] [PULL 48/50] char: use common error path in qmp_chardev_add,
Paolo Bonzini <=
- [Qemu-devel] [PULL 49/50] char: remove explicit_be_open from CharDriverState, Paolo Bonzini, 2016/10/24
- [Qemu-devel] [PULL 50/50] exec.c: workaround regression caused by alignment change in d2f39ad, Paolo Bonzini, 2016/10/24
- Re: [Qemu-devel] [PULL 00/50] Miscellaneous patches for 2016-10-24, Peter Maydell, 2016/10/24