check settings pointer and print error if null

ros-industrial · Sep 22, 2020 · c0be7eb · c0be7eb
1 parent db242bf
commit c0be7eb
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/socketcan_interface/include/socketcan_interface/socketcan.h b/socketcan_interface/include/socketcan_interface/socketcan.h
@@ -62,6 +62,10 @@ class SocketCANInterface : public AsioDriver<boost::asio::posix::stream_descript
         return init(device, loopback, NoSettings::create());
     }
     virtual bool init(const std::string &device, bool loopback, SettingsConstSharedPtr settings) override {
+      if (!settings) {
+          ROSCANOPEN_ERROR("socketcan_interface", "settings must not be a null pointer");
+          return false;
+      }
       const can_err_mask_t fatal_errors = ( CAN_ERR_TX_TIMEOUT   /* TX timeout (by netdevice driver) */
                                           | CAN_ERR_BUSOFF       /* bus off */
                                           | CAN_ERR_BUSERROR     /* bus error (may flood!) */