fixed memory leak

gilpazintel · Feb 18, 2025 · 7fb3959 · 7fb3959
1 parent 8ed6e12
commit 7fb3959
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/src/linux/backend-hid.cpp b/src/linux/backend-hid.cpp
@@ -760,7 +760,7 @@ namespace librealsense
             return false;
         }
 
-        bool iio_hid_sensor::sort_hids(hid_input* first, hid_input* second)
+        bool iio_hid_sensor::sort_hids(std::shared_ptr<hid_input> first, std::shared_ptr<hid_input> second)
         {
             return (second->get_hid_input_info().index >= first->get_hid_input_info().index);
         }
@@ -980,8 +980,9 @@ namespace librealsense
 
                         try
                         {
-                            auto* new_input = new hid_input(_iio_device_path, file);
+                            auto new_input = std::make_shared<hid_input>(_iio_device_path, file);
                             // push to input list.
+
                             _inputs.push_front(new_input);
                         }
                         catch(...)

diff --git a/src/linux/backend-hid.h b/src/linux/backend-hid.h
@@ -154,7 +154,7 @@ namespace librealsense
 
             bool has_metadata();
 
-            static bool sort_hids(hid_input* first, hid_input* second);
+            static bool sort_hids(std::shared_ptr<hid_input> first, std::shared_ptr<hid_input> second);
 
             void create_channel_array();
 
@@ -181,8 +181,8 @@ namespace librealsense
             std::string _sensor_name;
             std::string _sampling_frequency_name;
             std::string _sensitivity_name;
-            std::list<hid_input*> _inputs;
-            std::list<hid_input*> _channels;
+            std::list<std::shared_ptr<hid_input>> _inputs;
+            std::list<std::shared_ptr<hid_input>> _channels;
             hid_callback _callback;
             std::atomic<bool> _is_capturing;
             std::unique_ptr<std::thread> _hid_thread;