Merge pull request #159 from Habbus/fix/cancelreservation_segfault

c-jimenez · web-flow · commit 0573edf9b59f · 2023-10-19T12:46:49.000+02:00
Fix cancelreservation by passing connectorpointer by value instead of…
diff --git a/src/chargepoint/reservation/ReservationManager.cpp b/src/chargepoint/reservation/ReservationManager.cpp
@@ -296,7 +296,7 @@ bool ReservationManager::handleMessage(const ocpp::messages::CancelReservationRe
         if ((connector->status == ChargePointStatus::Reserved) && (connector->reservation_id == request.reservationId))
         {
             // Cancel reservation
-            m_worker_pool.run<void>([this, &connector] { endReservation(connector->id, true); });
+            m_worker_pool.run<void>([this, connector_id = connector->id] { endReservation(connector_id, true); });
 
             // Prepare response
             response.status = CancelReservationStatus::Accepted;

Original file line number	Diff line number	Diff line change
`@@ -296,7 +296,7 @@ bool ReservationManager::handleMessage(const ocpp::messages::CancelReservationRe`
`296`	`296`	`if ((connector->status == ChargePointStatus::Reserved) && (connector->reservation_id == request.reservationId))`
`297`	`297`	`{`
`298`	`298`	`// Cancel reservation`
`299`		`- m_worker_pool.run<void>([this, &connector] { endReservation(connector->id, true); });`
	`299`	`+ m_worker_pool.run<void>([this, connector_id = connector->id] { endReservation(connector_id, true); });`
`300`	`300`
`301`	`301`	`// Prepare response`
`302`	`302`	`response.status = CancelReservationStatus::Accepted;`