From: gichan <gichan2.jang@samsung.com>
Date: Tue, 4 Oct 2022 06:47:53 +0000 (+0900)
Subject: [Edge] Fix checking return value of the poll().
X-Git-Tag: accepted/tizen/unified/20221115.172904~27
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=f11f1eaa4a41af596b20ab7ff65eb77d928b7eda;p=platform%2Fupstream%2Fnnstreamer-edge.git

[Edge] Fix checking return value of the poll().

Since the timeout of the poll is set to 0, let's not check the return value of 0.

Signed-off-by: gichan <gichan2.jang@samsung.com>
---

diff --git a/src/libnnstreamer-edge/nnstreamer-edge-internal.c b/src/libnnstreamer-edge/nnstreamer-edge-internal.c
index 459d2a0..970b9b2 100644
--- a/src/libnnstreamer-edge/nnstreamer-edge-internal.c
+++ b/src/libnnstreamer-edge/nnstreamer-edge-internal.c
@@ -208,8 +208,13 @@ _nns_edge_check_connection (nns_edge_conn_s * conn)
   poll_fd.events = POLLIN | POLLOUT | POLLPRI | POLLERR | POLLHUP;
   poll_fd.revents = 0;
 
+  /** Timeout zero means that the poll() is returned immediately. */
   n = poll (&poll_fd, 1, 0);
-  if (n <= 0 || poll_fd.revents & (POLLERR | POLLHUP)) {
+  /**
+   * Return value zero indicates that the system call timed out.
+   * let's skip the check `n == 0` because timeout is set to 0.
+   */
+  if (n < 0 || poll_fd.revents & (POLLERR | POLLHUP)) {
     nns_edge_logw ("Socket is not available, possibly closed.");
     return false;
   }