Using EINA_LIST_FREE instead of EINA_LIST_FOREACH_SAFE.

To release the svc_ctx->packet_list.

Change-Id: I2a042ea14e5d11d06f74b724f879a85d9f5f2814

diff --git a/src/service_common.c b/src/service_common.c
index e94bf9f..6a32976 100644 (file)
--- a/src/service_common.c
+++ b/src/service_common.c
@@ -395,8 +395,6 @@ static void *server_main(void *data)
        struct tcb *tcb;
        int fd;
        char evt_ch;
-       Eina_List *l;
-       Eina_List *n;
        struct packet_info *packet_info;
 
        DbgPrint("Server thread is activated\n");
@@ -497,10 +495,9 @@ static void *server_main(void *data)
         * This only should be happenes while terminating the master daemon process.
         */
        CRITICAL_SECTION_BEGIN(&svc_ctx->packet_list_lock);
-       EINA_LIST_FOREACH_SAFE(svc_ctx->packet_list, l, n, packet_info) {
+       EINA_LIST_FREE(svc_ctx->packet_list, packet_info) {
                ret = read(svc_ctx->evt_pipe[PIPE_READ], &evt_ch, sizeof(evt_ch));
                DbgPrint("Flushing pipe: %d (%c)\n", ret, evt_ch);
-               svc_ctx->packet_list = eina_list_remove(svc_ctx->packet_list, packet_info);
                ret = svc_ctx->service_thread_main(packet_info->tcb, packet_info->packet, svc_ctx->service_thread_data);
                if (ret < 0)
                        ErrPrint("Service thread returns: %d\n", ret);