mavlink / MAVSDK

37%

main: 39%

LAST BUILD ON BRANCH refs/tags/v2.12.8

branch: refs/tags/v2.12.8

CHANGE BRANCH

x

Reset

• refs/tags/v2.12.8
• grpc2
• develop
• master
• v0.2.0
• fix-additional-includes
• add-vendor-product
• add-mavlink-statustexts
• fix-rc-status
• add-gimbal
• improve-debug-printfs
• fix-hover-test
• release-deb
• release-test-1
• follow_me
• add-mission-delay
• release-test-2
• improve-offboard
• hamishwillee-info_iscomplete
• mavlink-printfs
• add-lock
• grpc
• download-mission
• fix-android-build
• add-mock-time
• fix-exernal-example
• fix-tinyxml2
• vtol-fixes
• fix-addtional-includes
• fix-docs-generation
• fix-addtional-includes-test
• fake-time-test
• fix-result-str
• fix-doxygen-warnings
• fix-plugin-names
• build_system_minor_cleanup
• fix_grpc_build
• fix-serial-cpu
• fix-win-example-linking
• grpc-fixes
• improve-fly-mission
• improve-gendocs-sh
• fix-astyle-version
• lazy-init
• fix-telemetry-param
• dynamic-plugins
• fix-deb
• release-test-3
• release-test-4
• release-test-5
• message-debugging
• message-debugging-on
• improve-follow-me
• release-test-6
• release-test-7
• release-test-8
• release-test-9
• v1.2.3-test10
• v1.2.3-test11
• v1.2.3-test12
• v1.2.3-test13
• v1.2.3-test14
• v1.2.3-test15
• integration-tests-exe
• test-16
• test-17
• test-18
• test-19
• v1.2.3-test-20
• v1.2.3-test-21
• v1.2.3-test-23
• v1.2.3-test-24
• v1.2.3-test-25
• v1.2.3-test-26
• v1.2.3-test-27
• v0.3.0
• reorganize-grpc-server
• grpc-multiple-device
• fix-tcp-busyloop
• upstream3
• update-openssl
• follow_me_example
• pr
• fix-openssl-headers
• fix-openssl-headers2
• fix-minor-naming-issues
• update-mavlink
• fix-sitl-run-args
• fix-mission-download
• import-qgc-plan-mission
• fix-generate-doc
• fix-external
• hotfix-android-build
• 212-cross-compile-android
• fix-clean
• fix-android-tinyxml2-install
• fix-mac-install
• fix-docker-build
• fix-follow-me-boost-dep
• pr-jenkins
• fix-doxygen-code-snippets
• 238-refactor-dronecore-server
• import-qgc-plan-mission-rebase
• mac-static
• fix-ios
• 238-update-action-proto
• astyle-3-1
• doc_updates
• update-qgc-plan-path
• appveyor-add-rest-of-the-examples
• fix-mac-linking
• fix-hil-state
• appveyor-multiple-cores
• verify-multi-mission
• fix-camera-absolute-attitude
• try-clean-gtest-cmake
• remove-obsolete-test-flags
• run-tests-ci
• fix-dronecore-pimpl
• fix-android-json11
• refactor-connection-result
• connection_docs
• doc_globals
• 294-ios-backend-framework
• clean-pimpl
• rename-backend-dir
• fix-autopilot-type
• fix-takeoff-example
• multi-component-handling
• fix-reconnect-again
• fix-hil-set-mode
• parameterize-tests
• prefer-reference-over-ptr
• 267-update-telemetry-proto
• fix-backend
• move-action-result
• fix-protobuf-git-tag
• add-camera
• fix_gps
• debug-hack
• 268-update-mission-proto
• 269-update-core-proto
• add-ignore
• update-proto
• clear-old-mission-items
• 326-add-support-for-COMMAND_INT
• add-backend-packaging-scripts
• fix-apple-build
• add-ycm-conf
• notify-backend-started
• register-on-discover-lists-connected-systems
• core_improvements
• 333-telemetry-health
• reorganize-telemetry-structs
• add_camera
• 350-mission-backend
• fix-make-clean
• add-telemetry-features
• macos-serial
• v0.4.0
• finish-telemetry-features
• windows-serial
• add-action-features
• build-json11-static
• fix-warning-telemetry
• fix-mission-camera-action
• add-camera-proto
• fix-ios-build
• fix-bad-param-cast
• update-json11-commit
• build-tinyxml-statically
• fix-mission-lockup
• mission-progress
• blahtest
• mission-finished
• workaround-compid
• fix-sync-commands
• drop-armeabi
• fix-timeout-handler
• fix-connection-comments
• fix-mission-example
• add-thread-pool
• fix-various-camera-tests
• mission-paused
• mission-pause
• fix-warning
• fix-vtol-example
• system-refactor
• implement-camera-backend
• switch-to-clang-format
• fix-clang-format
• fix-mac-warning
• update-camera-interface
• implement-mission-backend
• fix-comments
• v0.5.0
• fix-camera-settings
• fix-jenkins-clang-format
• fix-json-null
• warn-mission-changes
• fix-backend-build
• camera-subscriptions
• fix-info-plist
• fix-lockup-again
• fix-camera-api
• add-calibration
• add-orbit
• 334-finish-camera-backend
• fix-readme
• rename
• add-plugin-warnings
• fix-readme-urls
• fix-docker-after-rename
• add-rtl-return-alt
• add-gimbal-calibration
• refactor-params
• header-cleanup
• fix-unused-variable
• fix-online-unit-tests
• fix-slowest-test
• fix-test-names
• update-dependencies
• fix-arming
• improve-long-param-methods
• fix-bashism
• fix-message-handler-clear
• fix-mission-origin
• fix-hitl-calibration
• fix-mac-ci
• no-color-for-ios
• fix-locked-queue
• embed-camera-defs
• fix-mission-upload-int-mode
• version-backend-framework-on-s3
• fix-mission-plan-path
• fix-jenkins-badge
• fix-cpp_rsc-install
• fix-install-path
• add-camera-description
• bump-backend-framework-0.0.3
• remove-boost-dependency
• support-multiple-vehicle-on-udp-port
• fix-camera-settings-descriptions-test
• add-sitl-ci
• fix-embedding
• bring-pybind-back
• add-setting-option-description
• bump-backend-framework-0.1.0
• fix-camera-option-description
• update-script-version
• fix-mission-progress-crash
• fix-mission-updates-again
• add-recording-time
• add-rtl-after-mission
• fix-mission-progress-tests
• add-flight-id
• fix-discovery
• camera-notify-after-subscribe
• update-proto-submodule
• add-waypoint-delay
• fix-armed-flickering
• add-recording-time-and-flight-id
• add-log-file-download
• add-set-rtl-altitude
• add-camera-euler-angles
• add-set-rtl-after-mission
• add-euler-angle-in-captureinfo
• bump-backend-0.2.0
• fix-notify-current-settings
• fix-camera-status-crash
• bump-backend-0.2.2
• add-version
• fix-race-in-camera
• add-fake-connection
• rename-backend
• bump-backend-0.2.4
• fix-settings-notification-again
• print-version
• remove-lock
• add-geofence
• add-release-script
• use-prebuilt
• build-backend-in-ci
• set-gimbal-yaw-absolute
• update-to-0.2.7
• fix-command-lockup
• pr-557-follow-up
• v0.12.0
• fix-travis
• fix-landed-state
• improve-integration-tests
• fix-plugin-headers
• update-fedora-docker
• add-mission-raw
• fix-plugin-init
• pr-docker_updates
• fix-mission-item-comparison
• v0.13.0
• update-core
• remove-jenkins
• remove-jenkins-badge
• add-mavlink-backdoor
• update-protos
• doc-fixes
• fix-framework-release-scripts
• v0.14.1
• fixing-tag-extraction
• fix-passthrough-reset
• fix-mission-callbacks
• fix-unregistering
• fix-mission-grpc-crash
• fix-camera-params
• fix-cmake
• alpine_dcsdk_docker
• add-example-multiple-vehicles
• fix-mission-protocol
• camera-improvements
• fix-werror
• change-discovery
• fix-action-goto-doc
• fix-camera-locks
• remove-termios2
• add-jenkins-again
• appveyor-move
• fix-pointer-not-aligned-ios
• add-configuration
• fix-mission-upload-cancel-crash
• fix-mission-double-call
• v0.15.0
• msvc-curl-linking
• reorganize-deps
• fix-long-param-names
• fix-msvc-dependent-name
• backend-with-any-connection
• fix-remove-backend-doesnot-unittests
• fix-passthrough-unsub
• v0.16.0
• fix-camera-threadsafety
• fix-ignore-qgc
• fix-gcc9-deprecated-copy
• fix-lossy-transfer
• improve-offboard-rate-control
• v0.17.0
• 758-fix-shared-libs-build
• cleanup
• patch-grpc
• install-backend-bin
• update-name-in-readme
• fix-build
• rename-mavsdk
• improve-travis-deployment
• v0.17.1
• v0.18.0
• fix-deployment
• update-ios-tools
• hamishwillee-readme
• fix-mavsdk-server-framework
• v0.18.1
• pr-add-logo
• 133-race-in-grpc
• v0.18.2
• fix-cross-compiling
• fix-clang-format-align
• fix-run-docker
• change-style
• pr-camera-range-params
• fix-ycm-conf
• pr-camera-deadlocks
• fix-style
• fix-integration-test
• v0.18.3
• 137-fix-camera-photo-interval
• update-grpc-to-1-22-0
• fix-health-position
• fix-jenkins-ci
• fix-serial-disconnect
• fix-mavlink-warning
• fix-kill
• fix-msg-rate
• add-gitlab-ci
• fix-offboard-ned-spelling
• fix-camera-setting-info
• fix-mission-reset
• fix-gimbal-comment
• remove-udp-bridge-filter
• camera-params-workaround
• 814-MSVC-mavsdk_server_bin
• 816-version-generated-files
• extract-cpp-rsc
• 836-protobuf-fails-rpi
• add-px4-tests
• v0.19.0
• fix-test-sleep
• improve-telemetry
• swap-json-dep
• fix-findtinyxml2
• fix-version-str
• v0.20.0
• enable-and-fix-geofence-server
• update-curl
• tune-plugin
• fix-zlib-patch
• cmake-remove-manylinux
• propagate-cmake-args
• update-grpc-1.24
• ios-travis-faster
• update-libressl-3.0.1
• fix-passthrough-integration-est
• clean-deps
• move-to-cpp17
• fix-loiter-before-arm
• no-werror
• rpi-in-travis
• remove-debug-appveyor
• publish-windows
• test-release-1
• fix-exe-name
• fix-win-exe-name
• v0.21.0
• v0.21.1
• generate-changelog
• remove-superfluous-params
• travis-ccache
• v0.21.2
• pr-add-rpm-deb
• test-upload-4
• test-upload-5
• test-upload-6
• v0.21.3
• v0.21.4
• v0.22.0
• v0.22.1
• v0.23.0
• v0.24.0
• HEAD
• refs/tags/v-test-10
• refs/tags/v0.25.0
• refs/tags/v0.26.0
• refs/tags/v0.27.0
• refs/tags/v0.28.0
• refs/tags/v0.29.0
• refs/tags/v0.30.0
• refs/tags/v0.30.1
• refs/tags/v0.31.0
• refs/tags/v0.32.0
• refs/tags/v0.32.1
• refs/tags/v0.33.0
• refs/tags/v-test-upload-2
• refs/tags/v-test-upload-3
• refs/tags/v0.33.1
• refs/tags/v0.34.0
• refs/tags/v0.35.0
• refs/tags/v0.35.1
• refs/tags/v0.36.0
• refs/tags/v0.37.0
• main
• refs/tags/v0.38.0
• refs/tags/v0.38.1
• refs/tags/v0.38.2
• refs/tags/v0.39.0
• refs/tags/v0.39.1
• refs/tags/v0.40.0
• refs/tags/v0.41.0
• refs/tags/v0.42.0
• refs/tags/v0.43.0
• refs/tags/v0.44.0
• refs/tags/v0.45.0
• refs/tags/v0.46.0
• refs/tags/v0.46.1
• refs/tags/v0.47.0
• refs/tags/v0.47.1
• refs/tags/v0.47.2
• refs/tags/v0.47.3
• refs/tags/v0.47.4
• refs/tags/v0.47.5
• refs/tags/v0.48.0
• refs/tags/v0.49.0
• refs/tags/v0.50.0
• refs/tags/v0.50.1
• refs/tags/v1.0.0
• refs/tags/v1.0.1
• refs/tags/v1.0.2
• refs/tags/v1.0.3
• refs/tags/v1.0.4
• refs/tags/v1.0.5
• refs/tags/v1.0.6
• refs/tags/v1.0.7
• refs/tags/v1.0.8
• refs/tags/v1.1.0
• refs/tags/v1.2.0
• refs/tags/v1.2.1
• refs/tags/v1.3.0
• refs/tags/v1.3.1
• refs/tags/v1.4.0
• refs/tags/v1.4.1
• refs/tags/v1.4.2
• refs/tags/v1.4.3
• refs/tags/v1.4.4
• refs/tags/v1.4.5
• refs/tags/v1.4.6
• refs/tags/v1.4.7
• refs/tags/v1.4.8
• refs/tags/v1.4.9
• refs/tags/v1.4.10
• refs/tags/v1.4.12
• refs/tags/v1.4.11
• refs/tags/v1.4.13
• refs/tags/v1.4.14
• refs/heads/main
• refs/tags/v1.4.15
• refs/tags/v1.4.16
• refs/tags/v1.4.17
• refs/tags/v1.4.18
• refs/tags/v2.0.0
• refs/tags/v2.0.1
• refs/tags/v2.1.0
• refs/tags/v2.2.0
• refs/tags/v2.3.0
• refs/tags/v2.4.0
• refs/tags/v2.4.1
• refs/tags/v2.5.0
• refs/tags/v2.5.1
• refs/tags/v2.6.0
• refs/tags/v2.7.0
• refs/tags/v2.8.0
• refs/tags/v2.8.1
• refs/tags/v2.9.0
• refs/tags/v2.9.1
• refs/tags/v2.10.0
• refs/tags/v2.10.1
• refs/tags/v2.10.2
• refs/tags/v2.11.0
• refs/tags/v2.12.0
• refs/tags/v2.12.1
• refs/tags/v2.12.2
• refs/tags/v2.12.3
• refs/tags/v2.12.4
• refs/tags/v2.12.5
• refs/tags/v2.12.6
• refs/tags/v2.12.7
• refs/tags/v2.12.9
• refs/tags/v2.12.10
• refs/tags/v2.12.11
• refs/tags/v2.12.12
• refs/tags/v2.13.0

Build # 11155262327

Build Type

push

github

Committed by web-flow

Commit Message

core: prevent lockup on connection destruction (#2412)

When we destroy Mavsdk and clear the list of connections, we likely
end up in a deadlock.

What happens is that:
1. A connection wants to forward a message and is trying to acquire the
   connection mutex.
2. At the same time, the connection is being destroyed, so we are
   waiting for the connection receive thread to be joinable. While the
   connections are being destroyed, we have the connection mutex which
   is blocking 1.

The proposed solution is to:
1. Make it less likely by acquiring the connection mutex properly before
   checking _connections.size() and not for the individual connections.
2. Check the _should_exit flag before trying to acquire the mutex. I
   believe by the time the connections are being cleared, this flag is
   set, and hence the deadlock should not happen, fingers crossed.

Run Details

5 of 13 new or added lines in 1 file covered. (38.46%)

3 existing lines in 1 file now uncovered.

11107 of 29722 relevant lines covered (37.37%)

255.2 hits per line