search

skupperproject / skupper-router

An application-layer router for Skupper networks
https://skupper.io
Apache License 2.0
14 stars 18 forks source link

Data race in system_tests_tcp_adaptor #1487

Closed ganeshmurthy closed 3 months ago

ganeshmurthy commented 3 months ago 
61: WARNING: ThreadSanitizer: data race (pid=7628)
61:   Write of size 1 at 0x7b4800025dc0 by thread T3:
61:     #0 qd_dispatch_delete_tcp_connector ../src/adaptors/tcp/tcp_adaptor.c:2418 (skrouterd+0x6e969) (BuildId: 2adccfbb4fcb6f58aaf32414048bb83fef28f2c5)
61:     #1 <null> <null> (libffi.so.8+0x7e2d) (BuildId: 59c2a6b204f74f358ca7711d2dfd349d88711f6a)
61:     #2 qdr_forward_on_message ../src/router_core/forwarder.c:375 (skrouterd+0x10163b) (BuildId: 2adccfbb4fcb6f58aaf32414048bb83fef28f2c5)
61:     #3 qdr_general_handler ../src/router_core/router_core.c:1032 (skrouterd+0xfee4b) (BuildId: 2adccfbb4fcb6f58aaf32414048bb83fef28f2c5)
61:     #4 qd_timer_visit ../src/timer.c:317 (skrouterd+0x136869) (BuildId: 2adccfbb4fcb6f58aaf32414048bb83fef28f2c5)
61:     #5 handle_proactor_other_event ../src/server.c:142 (skrouterd+0x13263c) (BuildId: 2adccfbb4fcb6f58aaf32414048bb83fef28f2c5)
61:     #6 proactor_thread ../src/server.c:194 (skrouterd+0x120672) (BuildId: 2adccfbb4fcb6f58aaf32414048bb83fef28f2c5)
61:     #7 _thread_init ../src/posix/threading.c:207 (skrouterd+0xb70f1) (BuildId: 2adccfbb4fcb6f58aaf32414048bb83fef28f2c5)
61: 
61:   Previous read of size 1 at 0x7b4800025dc0 by thread T5 (mutexes: write M0):
61:     #0 free_connection_IO ../src/adaptors/tcp/tcp_adaptor.c:476 (skrouterd+0x6e2e3) (BuildId: 2adccfbb4fcb6f58aaf32414048bb83fef28f2c5)
61:     #1 close_connection_XSIDE_IO ../src/adaptors/tcp/tcp_adaptor.c:615 (skrouterd+0x6e2e3)
61:     #2 on_connection_event_CSIDE_IO ../src/adaptors/tcp/tcp_adaptor.c:1938 (skrouterd+0x7458a) (BuildId: 2adccfbb4fcb6f58aaf32414048bb83fef28f2c5)
61:     #3 handle_event_with_context ../src/server.c:107 (skrouterd+0x12050d) (BuildId: 2adccfbb4fcb6f58aaf32414048bb83fef28f2c5)
61:     #4 handle_raw_connection_event ../src/server.c:115 (skrouterd+0x12050d)
61:     #5 handle_raw_connection_event ../src/server.c:111 (skrouterd+0x12050d)
61:     #6 proactor_thread ../src/server.c:194 (skrouterd+0x120672) (BuildId: 2adccfbb4fcb6f58aaf32414048bb83fef28f2c5)
61:     #7 _thread_init ../src/posix/threading.c:207 (skrouterd+0xb70f1) (BuildId: 2adccfbb4fcb6f58aaf32414048bb83fef28f2c5)
61: 
61:   Location is heap block of size 384 at 0x7b4800025c80 allocated by thread T3:
61:     #0 posix_memalign ../../../../src/libsanitizer/tsan/tsan_interceptors_posix.cpp:873 (libtsan.so.2+0x3d087) (BuildId: 5e151fac359cd43a07192270fb85f74e380f2fc8)
61:     #1 qd_alloc ../src/alloc_pool.c:348 (skrouterd+0x8cb42) (BuildId: 2adccfbb4fcb6f58aaf32414048bb83fef28f2c5)
61:     #2 new_qd_tcp_connector_t ../src/adaptors/tcp/tcp_adaptor.c:50 (skrouterd+0x76602) (BuildId: 2adccfbb4fcb6f58aaf32414048bb83fef28f2c5)
61:     #3 qd_dispatch_configure_tcp_connector ../src/adaptors/tcp/tcp_adaptor.c:2363 (skrouterd+0x76602)
61:     #4 <null> <null> (libffi.so.8+0x7e2d) (BuildId: 59c2a6b204f74f358ca7711d2dfd349d88711f6a)
61:     #5 qdr_forward_on_message ../src/router_core/forwarder.c:375 (skrouterd+0x10163b) (BuildId: 2adccfbb4fcb6f58aaf32414048bb83fef28f2c5)
61:     #6 qdr_general_handler ../src/router_core/router_core.c:1032 (skrouterd+0xfee4b) (BuildId: 2adccfbb4fcb6f58aaf32414048bb83fef28f2c5)
61:     #7 qd_timer_visit ../src/timer.c:317 (skrouterd+0x136869) (BuildId: 2adccfbb4fcb6f58aaf32414048bb83fef28f2c5)
61:     #8 handle_proactor_other_event ../src/server.c:142 (skrouterd+0x13263c) (BuildId: 2adccfbb4fcb6f58aaf32414048bb83fef28f2c5)
61:     #9 proactor_thread ../src/server.c:194 (skrouterd+0x120672) (BuildId: 2adccfbb4fcb6f58aaf32414048bb83fef28f2c5)
61:     #10 _thread_init ../src/posix/threading.c:207 (skrouterd+0xb70f1) (BuildId: 2adccfbb4fcb6f58aaf32414048bb83fef28f2c5)
61: 
61:   Mutex M0 (0x7b4800025d38) created at:
61:     #0 pthread_mutex_init ../../../../src/libsanitizer/tsan/tsan_interceptors_posix.cpp:1329 (libtsan.so.2+0x3d7a6) (BuildId: 5e151fac359cd43a07192270fb85f74e380f2fc8)
61:     #1 sys_mutex_init ../src/posix/threading.c:40 (skrouterd+0x76787) (BuildId: 2adccfbb4fcb6f58aaf32414048bb83fef28f2c5)
61:     #2 qd_dispatch_configure_tcp_connector ../src/adaptors/tcp/tcp_adaptor.c:2399 (skrouterd+0x76787)
61:     #3 <null> <null> (libffi.so.8+0x7e2d) (BuildId: 59c2a6b204f74f358ca7711d2dfd349d88711f6a)
61:     #4 qdr_forward_on_message ../src/router_core/forwarder.c:375 (skrouterd+0x10163b) (BuildId: 2adccfbb4fcb6f58aaf32414048bb83fef28f2c5)
61:     #5 qdr_general_handler ../src/router_core/router_core.c:1032 (skrouterd+0xfee4b) (BuildId: 2adccfbb4fcb6f58aaf32414048bb83fef28f2c5)
61:     #6 qd_timer_visit ../src/timer.c:317 (skrouterd+0x136869) (BuildId: 2adccfbb4fcb6f58aaf32414048bb83fef28f2c5)
61:     #7 handle_proactor_other_event ../src/server.c:142 (skrouterd+0x13263c) (BuildId: 2adccfbb4fcb6f58aaf32414048bb83fef28f2c5)
61:     #8 proactor_thread ../src/server.c:194 (skrouterd+0x120672) (BuildId: 2adccfbb4fcb6f58aaf32414048bb83fef28f2c5)
61:     #9 _thread_init ../src/posix/threading.c:207 (skrouterd+0xb70f1) (BuildId: 2adccfbb4fcb6f58aaf32414048bb83fef28f2c5)
61: 
61:   Thread T3 'wrkr_0' (tid=7634, running) created by main thread at:
61:     #0 pthread_create ../../../../src/libsanitizer/tsan/tsan_interceptors_posix.cpp:1036 (libtsan.so.2+0x3d179) (BuildId: 5e151fac359cd43a07192270fb85f74e380f2fc8)
61:     #1 sys_thread ../src/posix/threading.c:229 (skrouterd+0xbacf4) (BuildId: 2adccfbb4fcb6f58aaf32414048bb83fef28f2c5)
61:     #2 qd_server_run ../src/server.c:298 (skrouterd+0x136dc3) (BuildId: 2adccfbb4fcb6f58aaf32414048bb83fef28f2c5)
61:     #3 main_process ../router/src/main.c:111 (skrouterd+0x137e60) (BuildId: 2adccfbb4fcb6f58aaf32414048bb83fef28f2c5)
61:     #4 main ../router/src/main.c:365 (skrouterd+0x3160e) (BuildId: 2adccfbb4fcb6f58aaf32414048bb83fef28f2c5)
61: 
61:   Thread T5 'wrkr_2' (tid=7636, running) created by main thread at:
61:     #0 pthread_create ../../../../src/libsanitizer/tsan/tsan_interceptors_posix.cpp:1036 (libtsan.so.2+0x3d179) (BuildId: 5e151fac359cd43a07192270fb85f74e380f2fc8)
61:     #1 sys_thread ../src/posix/threading.c:229 (skrouterd+0xbacf4) (BuildId: 2adccfbb4fcb6f58aaf32414048bb83fef28f2c5)
61:     #2 qd_server_run ../src/server.c:298 (skrouterd+0x136dc3) (BuildId: 2adccfbb4fcb6f58aaf32414048bb83fef28f2c5)
61:     #3 main_process ../router/src/main.c:111 (skrouterd+0x137e60) (BuildId: 2adccfbb4fcb6f58aaf32414048bb83fef28f2c5)
61:     #4 main ../router/src/main.c:365 (skrouterd+0x3160e) (BuildId: 2adccfbb4fcb6f58aaf32414048bb83fef28f2c5)

https://github.com/skupperproject/skupper-router/actions/runs/8834188946/job/24255451855#step:10:4955