Fix memory and resource leak via user callback management

Remove unused variables

src/pipeline/resolver.h

@@ -288,7 +288,6 @@ namespace librealsense
 
                 // Unpack the data returned by assign
                 std::map<int, stream_profiles> dev_to_profiles;
-                std::map<index_type, sensor_interface*> stream_to_dev;
                 std::map<index_type, std::shared_ptr<stream_profile_interface>> stream_to_profile;
 
                 std::map<int, sensor_interface*> sensors_map;
@@ -303,7 +302,6 @@ namespace librealsense
                 for (auto && kvp : mapping) {
                     dev_to_profiles[kvp.first].push_back(kvp.second);
                     index_type idx{ kvp.second->get_stream_type(), kvp.second->get_stream_index() };
-                    stream_to_dev.emplace(idx, sensors_map.at(kvp.first));
                     stream_to_profile[idx] = kvp.second;
                 }
 

src/win7/win7-uvc.cpp

@@ -610,14 +610,14 @@ namespace librealsense
 
             winusb_free_formats(formats);
 
-            callback_context *context = new callback_context();
-            context->_callback = callback;
-            context->_this = this;
-            context->_profile = profile;
-
-            auto sts = winusb_start_streaming(_device.get(), &ctrl, internal_winusb_uvc_callback, context, 0);
-			if(sts != UVC_SUCCESS)
-				throw std::runtime_error("Failed to start streaming!");
+            _cb_context = std::make_shared<callback_context>();
+            _cb_context->_callback = callback;
+            _cb_context->_profile = profile;
+            _cb_context->_this = this;
+
+            auto sts = winusb_start_streaming(_device.get(), &ctrl, internal_winusb_uvc_callback, _cb_context.get(), 0);
+            if(sts != UVC_SUCCESS)
+                throw std::runtime_error("Failed to start streaming!");
         }
 
         void win7_uvc_device::stream_on(std::function<void(const notification& n)> error_handler)

src/win7/win7-uvc.h

@@ -30,6 +30,7 @@ namespace librealsense
     namespace platform
     {
         class win7_backend;
+        struct callback_context;
 
         struct profile_and_callback
         {
@@ -99,6 +100,7 @@ namespace librealsense
             std::mutex                              _streams_mutex;
 
             std::shared_ptr<const win7_backend>      _backend;
+            std::shared_ptr<callback_context>        _cb_context;
 
             std::string                             _location;
             usb_spec                                _device_usb_spec;

src/win7/winusb_uvc/winusb_uvc.cpp

@@ -1115,7 +1115,6 @@ void winusb_uvc_stream_close(winusb_uvc_stream_handle_t *strmh)
 
 	DL_DELETE(strmh->devh->streams, strmh);
 
-	free(strmh->user_ptr);
 	free(strmh);
 }
 

src/win7/winusb_uvc/winusb_uvc.h

@@ -470,7 +470,7 @@ struct winusb_uvc_stream_handle {
 
     std::thread cb_thread;
     winusb_uvc_frame_callback_t *user_cb;
-    void *user_ptr;
+    void *user_ptr;                         // User-allocated memory
     enum uvc_frame_format frame_format;
 };