From 4105394234536f8f41276d1b7a4b783fffde31b1 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Beat=20K=C3=BCng?= Date: Fri, 12 May 2017 11:00:29 +0200 Subject: [PATCH] mavlink_receiver: check for allocation failure Also remove the start() declration (there's no definition of that) --- src/modules/mavlink/mavlink_receiver.cpp | 5 +++++ src/modules/mavlink/mavlink_receiver.h | 7 ------- 2 files changed, 5 insertions(+), 7 deletions(-) diff --git a/src/modules/mavlink/mavlink_receiver.cpp b/src/modules/mavlink/mavlink_receiver.cpp index 8bcaf22c4b..fbefda08fd 100644 --- a/src/modules/mavlink/mavlink_receiver.cpp +++ b/src/modules/mavlink/mavlink_receiver.cpp @@ -2493,6 +2493,11 @@ void *MavlinkReceiver::start_helper(void *context) MavlinkReceiver *rcv = new MavlinkReceiver((Mavlink *)context); + if (!rcv) { + PX4_ERR("alloc failed"); + return nullptr; + } + void *ret = rcv->receive_thread(nullptr); delete rcv; diff --git a/src/modules/mavlink/mavlink_receiver.h b/src/modules/mavlink/mavlink_receiver.h index 7b144f50c8..9315a12fee 100644 --- a/src/modules/mavlink/mavlink_receiver.h +++ b/src/modules/mavlink/mavlink_receiver.h @@ -99,13 +99,6 @@ public: */ ~MavlinkReceiver(); - /** - * Start the mavlink task. - * - * @return OK on success. - */ - int start(); - /** * Display the mavlink status. */