rtr: fix race condition on rtr_start/rtr_close#273
rtr: fix race condition on rtr_start/rtr_close#273mroethke wants to merge 1 commit intortrlib:masterfrom
Conversation
It was possible for two threads to be started for the same socket because the check supposed to prevent this used an unsynchronized variable.
|
@Jaykov-s please confirm that this fixes the problem |
Continuous Integration Result: FAILEDSee below for issues. This is a comment from an automated CI system. Get source / Pull Request: SuccessfulBuilding Stage: FailedDebian 11 amd64: Failed (click for details)Debian 11 amd64: No useful log foundSuccessful on other platforms/tests
Warnings Generated during build:Ubuntu 18.04 i386: Successful with additional warningsDebian Package lintian failed for Ubuntu 18.04 i386: Ubuntu 18.04 amd64: Successful with additional warningsDebian Package lintian failed for Ubuntu 18.04 amd64: Debian 9 arm7: Successful with additional warningsDebian Package lintian failed for Debian 9 arm7: Ubuntu 16.04 arm8: Successful with additional warningsDebian Package lintian failed for Ubuntu 16.04 arm8: Debian 9 arm8: Successful with additional warningsDebian Package lintian failed for Debian 9 arm8: Debian 10 amd64: Successful with additional warningsDebian Package lintian failed for Debian 10 amd64: Ubuntu 18.04 ppc64le: Successful with additional warningsDebian Package lintian failed for Ubuntu 18.04 ppc64le: Ubuntu 18.04 arm8: Successful with additional warningsDebian Package lintian failed for Ubuntu 18.04 arm8: Ubuntu 16.04 amd64: Successful with additional warningsDebian Package lintian failed for Ubuntu 16.04 amd64: Ubuntu 16.04 arm7: Successful with additional warningsDebian Package lintian failed for Ubuntu 16.04 arm7: Ubuntu 16.04 i386: Successful with additional warningsDebian Package lintian failed for Ubuntu 16.04 i386: Ubuntu 18.04 arm7: Successful with additional warningsDebian Package lintian failed for Ubuntu 18.04 arm7: Ubuntu 14.04 amd64: Successful with additional warningsDebian Package lintian failed for Ubuntu 14.04 amd64: Ubuntu 20.04 arm8: Successful with additional warningsDebian Package lintian failed for Ubuntu 20.04 arm8: Debian 9 amd64: Successful with additional warningsDebian Package lintian failed for Debian 9 amd64: Debian 8 amd64: Successful with additional warningsDebian Package lintian failed for Debian 8 amd64: |
|
CI:rerun |
Continuous Integration Result: SUCCESSFULCongratulations, this patch passed basic tests Tested-by: NetDEF / OpenSourceRouting.org CI System CI System Testrun URL: https://ci1.netdef.org/browse/RPKI-RTRLIBPR-395/ This is a comment from an automated CI system. Warnings Generated during build:Ubuntu 18.04 amd64: Successful with additional warningsDebian Package lintian failed for Ubuntu 18.04 amd64: Ubuntu 18.04 i386: Successful with additional warningsDebian Package lintian failed for Ubuntu 18.04 i386: Ubuntu 18.04 ppc64le: Successful with additional warningsDebian Package lintian failed for Ubuntu 18.04 ppc64le: Debian 11 amd64: Successful with additional warningsDebian Package lintian failed for Debian 11 amd64: Ubuntu 16.04 arm7: Successful with additional warningsDebian Package lintian failed for Ubuntu 16.04 arm7: Debian 10 amd64: Successful with additional warningsDebian Package lintian failed for Debian 10 amd64: Debian 9 arm7: Successful with additional warningsDebian Package lintian failed for Debian 9 arm7: Ubuntu 16.04 arm8: Successful with additional warningsDebian Package lintian failed for Ubuntu 16.04 arm8: Debian 9 arm8: Successful with additional warningsDebian Package lintian failed for Debian 9 arm8: Ubuntu 16.04 amd64: Successful with additional warningsDebian Package lintian failed for Ubuntu 16.04 amd64: Debian 9 amd64: Successful with additional warningsDebian Package lintian failed for Debian 9 amd64: Ubuntu 16.04 i386: Successful with additional warningsDebian Package lintian failed for Ubuntu 16.04 i386: Ubuntu 18.04 arm7: Successful with additional warningsDebian Package lintian failed for Ubuntu 18.04 arm7: Ubuntu 14.04 amd64: Successful with additional warningsDebian Package lintian failed for Ubuntu 14.04 amd64: Ubuntu 20.04 arm8: Successful with additional warningsDebian Package lintian failed for Ubuntu 20.04 arm8: Ubuntu 18.04 arm8: Successful with additional warningsDebian Package lintian failed for Ubuntu 18.04 arm8: Debian 8 amd64: Successful with additional warningsDebian Package lintian failed for Debian 8 amd64: |
|
@mroethke Hi, it's seem that it solve the issue. Thanks. |
Nimuedaertya
left a comment
There was a problem hiding this comment.
Tested and looked at the code. Seems good.
|
CI:rerun |
It was possible for two threads to be started for the same socket
because the check supposed to prevent this used an unsynchronized
variable.
Fix #271