Skip to content

Commit

Permalink
land_detector: Remove recently added vehicle_attitude.timestamp check…
Browse files Browse the repository at this point in the history
… from MulticopterLandDetector::_get_maybe_landed_state(). (#13072)

PR #12681 added a check to the MulticopterLandDetector::_get_maybe_landed_state() method for a valid vehicle_attitude.timstamp value to finish work in PR #9756. It was discovered that the addition of that check leaves it possible to fly in acro mode without a valid attitude and auto-disarm can engage, allowing the multicopter to fall out of the sky.
  • Loading branch information
mcsauder authored and dagar committed Oct 2, 2019
1 parent d537f3e commit 508ffa1
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/modules/land_detector/MulticopterLandDetector.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -178,7 +178,7 @@ bool MulticopterLandDetector::_get_maybe_landed_state()
const hrt_abstime now = hrt_absolute_time();

// When not armed, consider to be maybe-landed
if (!_actuator_armed.armed || (_vehicle_attitude.timestamp == 0)) {
if (!_actuator_armed.armed) {
return true;
}

Expand Down

0 comments on commit 508ffa1

Please sign in to comment.