From e87d8f60fa9b6eaa6a2357545a96e4fff05dbef0 Mon Sep 17 00:00:00 2001 From: Julien Grall Date: Thu, 24 Jun 2021 15:55:03 +0100 Subject: [PATCH] tools/xenstored: Remove redundant check in socket_can_process() Commit 3adfb50315d9 ("tools/xenstored: Introduce a wrapper for conn->funcs->can_{read, write}") consolidated the check !conn->is_ignored in two new wrappers. This means the check in socket_can_process() is now redundant. In fact it should have been removed in orignal commit (as it was done for the domain helpers). Reported-by: Raphael Ning Reviewed-by: Juergen Gross --- tools/xenstore/xenstored_core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/xenstore/xenstored_core.c b/tools/xenstore/xenstored_core.c index 9ffd2ac66d..cf7297a96c 100644 --- a/tools/xenstore/xenstored_core.c +++ b/tools/xenstore/xenstored_core.c @@ -1752,7 +1752,7 @@ static bool socket_can_process(struct connection *conn, int mask) return false; } - return (fds[conn->pollfd_idx].revents & mask) && !conn->is_ignored; + return (fds[conn->pollfd_idx].revents & mask); } static bool socket_can_write(struct connection *conn) -- 2.39.5