Merge pull request #122 from TheSayOL/fix_poll

fix a bug for sys_poll and sys_ppoll
syswonder · Jun 6, 2024 · f841b85 · f841b85
2 parents 027391d + 5030322
commit f841b85
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/api/ruxos_posix_api/src/imp/io_mpx/poll.rs b/api/ruxos_posix_api/src/imp/io_mpx/poll.rs
@@ -24,19 +24,21 @@ fn poll_all(fds: &mut [ctypes::pollfd]) -> LinuxResult<usize> {
             Err(_) => {
                 if (events & ctypes::EPOLLERR as i16) != 0 {
                     *revents |= ctypes::EPOLLERR as i16;
+                    events_num += 1;
                 }
             }
             Ok(state) => {
                 if state.readable && (events & ctypes::EPOLLIN as i16 != 0) {
                     *revents |= ctypes::EPOLLIN as i16;
+                    events_num += 1;
                 }
 
                 if state.writable && (events & ctypes::EPOLLOUT as i16 != 0) {
                     *revents |= ctypes::EPOLLOUT as i16;
+                    events_num += 1;
                 }
             }
         }
-        events_num += 1;
     }
     Ok(events_num)
 }