From: ye xingchen <ye.xingc...@zte.com.cn>

Return the value epoll_ctl() directly instead of storing it in another
redundant variable.

Reported-by: Zeal Robot <zea...@zte.com.cn>
Signed-off-by: ye xingchen <ye.xingc...@zte.com.cn>
---
 arch/um/os-Linux/irq.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/arch/um/os-Linux/irq.c b/arch/um/os-Linux/irq.c
index 98ea910ef87c..cf7e49c08b21 100644
--- a/arch/um/os-Linux/irq.c
+++ b/arch/um/os-Linux/irq.c
@@ -127,12 +127,10 @@ int os_mod_epoll_fd(int events, int fd, void *data)
 int os_del_epoll_fd(int fd)
 {
        struct epoll_event event;
-       int result;
        /* This is quiet as we use this as IO ON/OFF - so it is often
         * invoked on a non-existent fd
         */
-       result = epoll_ctl(epollfd, EPOLL_CTL_DEL, fd, &event);
-       return result;
+       return epoll_ctl(epollfd, EPOLL_CTL_DEL, fd, &event);
 }
 
 void os_set_ioignore(void)
-- 
2.25.1

_______________________________________________
linux-um mailing list
linux-um@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-um

Reply via email to