aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorToni Uhlig <matzeton@googlemail.com>2022-10-22 00:25:39 +0200
committerToni Uhlig <matzeton@googlemail.com>2022-10-22 01:45:14 +0200
commit916d2df6eac8acdd2329ba12897eae21d2ca7c87 (patch)
treecdfb20b1a8b0f375db980051e9cd8a2c21468c56
parent46c8fc521912e91f44a4f0ebd442209a071dbdff (diff)
nDPId-test: Fixed thread sync/lock issue.
* rarely happens in CI Signed-off-by: Toni Uhlig <matzeton@googlemail.com>
-rw-r--r--.gitlab-ci.yml4
-rw-r--r--nDPId-test.c5
2 files changed, 7 insertions, 2 deletions
diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index 41ad3109d..d763fbba1 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -15,8 +15,8 @@ before_script:
netcat-openbsd python3 python3-jsonschema tree lcov
after_script:
- - cat /tmp/nDPIsrvd.log
- - cat /tmp/nDPId.log
+ - test -r && cat /tmp/nDPIsrvd.log
+ - test -r && cat /tmp/nDPId.log
build_and_test:
script:
diff --git a/nDPId-test.c b/nDPId-test.c
index 10bde9358..f7260edcf 100644
--- a/nDPId-test.c
+++ b/nDPId-test.c
@@ -293,6 +293,7 @@ error:
drain_write_buffers_blocking(mock_arpa_desc);
}
+ pthread_mutex_lock(&nDPIsrvd_start_mutex);
free_remotes(epollfd);
close(epollfd);
@@ -415,6 +416,9 @@ static enum nDPIsrvd_callback_return distributor_json_callback(struct nDPIsrvd_s
global_stats->total_events_serialized = nmb;
}
+
+ pthread_mutex_unlock(&nDPIsrvd_start_mutex);
+ pthread_mutex_unlock(&nDPId_start_mutex);
}
}
}
@@ -853,6 +857,7 @@ static void * nDPId_mainloop_thread(void * const arg)
}
error:
+ pthread_mutex_lock(&nDPId_start_mutex);
free_reader_threads();
close(mock_pipefds[PIPE_nDPId]);