daemon: Loop for client fds

In dlt_daemon_client_send_all_multiple(), while in the loop,
the connection to client may be destroyed by recursive call to
dlt_daemon_close_socket(). So in some scenario we still use the removed
connection, which is unexpected behavior.

Solution: loop on the fds array and find the client connection
corresponds to.

Signed-off-by: Vo Trung Chi <Chi.VoTrung@vn.bosch.com>

1 files changed, 5 insertions, 4 deletions

diff --git a/src/daemon/dlt_daemon_event_handler.c b/src/daemon/dlt_daemon_event_handler.c
index 895a896..38aa022 100644
--- a/src/daemon/dlt_daemon_event_handler.c
+++ b/src/daemon/dlt_daemon_event_handler.c
@@ -285,14 +285,15 @@ int dlt_daemon_handle_event(DltEventHandler *pEvent,
  *
  * @return The found connection pointer, NULL otherwise.
  */
-DltConnection *dlt_event_handler_find_connection(DltEventHandler *ev,
-                                                 int fd)
+DltConnection *dlt_event_handler_find_connection(DltEventHandler *ev, int fd)
 {
-
     DltConnection *temp = ev->connections;
 
-    while ((temp != NULL) && (temp->receiver->fd != fd))
+    while (temp != NULL) {
+        if ((temp->receiver != NULL) && (temp->receiver->fd == fd))
+            return temp;
         temp = temp->next;
+    }
 
     return temp;
 }