From bef8b6b94996b3528630e3ec19a110df3e849765 Mon Sep 17 00:00:00 2001 From: Roman Bapst Date: Sun, 25 Nov 2018 20:01:23 +0100 Subject: [PATCH] mavlink hash check: return early in case of hash check parameter (#10861) * a request for setting the '_HASH_CHECK' parameter is handled differently than the same request for a standard parameter. Make sure we don't actually try to set the parameter. Signed-off-by: Roman --- src/modules/mavlink/mavlink_parameters.cpp | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/src/modules/mavlink/mavlink_parameters.cpp b/src/modules/mavlink/mavlink_parameters.cpp index b853648c22..cbfdf75f2d 100644 --- a/src/modules/mavlink/mavlink_parameters.cpp +++ b/src/modules/mavlink/mavlink_parameters.cpp @@ -134,8 +134,12 @@ MavlinkParametersManager::handle_message(const mavlink_message_t *msg) name[MAVLINK_MSG_PARAM_VALUE_FIELD_PARAM_ID_LEN] = '\0'; /* Whatever the value is, we're being told to stop sending */ - if (strncmp(name, "_HASH_CHECK", sizeof(name)) == 0 && _mavlink->hash_check_enabled()) { - _send_all_index = -1; + if (strncmp(name, "_HASH_CHECK", sizeof(name)) == 0) { + + if (_mavlink->hash_check_enabled()) { + _send_all_index = -1; + } + /* No other action taken, return */ return; }