-
Notifications
You must be signed in to change notification settings - Fork 37
rtr: fix race condition on rtr_start/rtr_close #273
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
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. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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