[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 04/10] monitor: Simplify monitor_fd_param()'s error handling
From: |
Markus Armbruster |
Subject: |
[PATCH 04/10] monitor: Simplify monitor_fd_param()'s error handling |
Date: |
Mon, 21 Nov 2022 09:50:48 +0100 |
Cc: Dr. David Alan Gilbert <dgilbert@redhat.com>
Signed-off-by: Markus Armbruster <armbru@redhat.com>
---
monitor/misc.c | 14 ++++----------
1 file changed, 4 insertions(+), 10 deletions(-)
diff --git a/monitor/misc.c b/monitor/misc.c
index 205487e2b9..83d7f4ffde 100644
--- a/monitor/misc.c
+++ b/monitor/misc.c
@@ -1086,6 +1086,7 @@ int monitor_get_fd(Monitor *mon, const char *fdname,
Error **errp)
}
fd = monfd->fd;
+ assert(fd >= 0);
/* caller takes ownership of fd */
QLIST_REMOVE(monfd, next);
@@ -1403,23 +1404,16 @@ void monitor_fdset_dup_fd_remove(int dup_fd)
int monitor_fd_param(Monitor *mon, const char *fdname, Error **errp)
{
int fd;
- Error *local_err = NULL;
if (!qemu_isdigit(fdname[0]) && mon) {
- fd = monitor_get_fd(mon, fdname, &local_err);
+ fd = monitor_get_fd(mon, fdname, errp);
} else {
fd = qemu_parse_fd(fdname);
- if (fd == -1) {
- error_setg(&local_err, "Invalid file descriptor number '%s'",
+ if (fd < 0) {
+ error_setg(errp, "Invalid file descriptor number '%s'",
fdname);
}
}
- if (local_err) {
- error_propagate(errp, local_err);
- assert(fd == -1);
- } else {
- assert(fd != -1);
- }
return fd;
}
--
2.37.3
- [PATCH 00/10] Cleanups around error_propagate() and ERRP_GUARD(), Markus Armbruster, 2022/11/21
- [PATCH 03/10] error: Move ERRP_GUARD() to the beginning of the function, Markus Armbruster, 2022/11/21
- [PATCH 04/10] monitor: Simplify monitor_fd_param()'s error handling,
Markus Armbruster <=
- [PATCH 08/10] sockets: Use ERRP_GUARD() where obviously appropriate, Markus Armbruster, 2022/11/21
- [PATCH 02/10] error: Drop a few superfluous ERRP_GUARD(), Markus Armbruster, 2022/11/21
- [PATCH 07/10] qemu-config: Use ERRP_GUARD() where obviously appropriate, Markus Armbruster, 2022/11/21
- [PATCH 09/10] qapi: Use returned bool to check for failure (again), Markus Armbruster, 2022/11/21
- [PATCH 10/10] io: Tidy up fat-fingered parameter name, Markus Armbruster, 2022/11/21
- [PATCH 05/10] monitor: Use ERRP_GUARD() in monitor_init(), Markus Armbruster, 2022/11/21
- [PATCH 06/10] qemu-config: Make config_parse_qdict() return bool, Markus Armbruster, 2022/11/21
- [PATCH 01/10] error: Drop some obviously superfluous error_propagate(), Markus Armbruster, 2022/11/21