Mismatched free fix (#399)

Co-authored-by: Jack-ReframeSystems <jack@reframe.systems>
tesseract-robotics · Apr 13, 2024 · 5aec94c · 5aec94c
1 parent 87f25da
commit 5aec94c
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/trajopt_sco/include/trajopt_sco/osqp_interface.hpp b/trajopt_sco/include/trajopt_sco/osqp_interface.hpp
@@ -60,12 +60,12 @@ class OSQPModel : public Model
   ConstraintTypeVector cnt_types_; /**< constraints types */
   DblVec solution_;                /**< optimizizer's solution for current model */
 
-  std::unique_ptr<csc> P_;               /**< Takes ownership of OSQPData.P to avoid having to deallocate manually */
-  std::unique_ptr<csc> A_;               /**< Takes ownership of OSQPData.A to avoid having to deallocate manually */
-  std::vector<c_int> P_row_indices_;     /**< row indices for P, CSC format */
-  std::vector<c_int> P_column_pointers_; /**< column pointers for P, CSC format */
-  DblVec P_csc_data_;                    /**< P values in CSC format */
-  Eigen::VectorXd q_;                    /**< linear part of the objective */
+  std::unique_ptr<csc, decltype(&free)> P_; /**< Takes ownership of OSQPData.P to avoid having to deallocate manually */
+  std::unique_ptr<csc, decltype(&free)> A_; /**< Takes ownership of OSQPData.A to avoid having to deallocate manually */
+  std::vector<c_int> P_row_indices_;        /**< row indices for P, CSC format */
+  std::vector<c_int> P_column_pointers_;    /**< column pointers for P, CSC format */
+  DblVec P_csc_data_;                       /**< P values in CSC format */
+  Eigen::VectorXd q_;                       /**< linear part of the objective */
 
   std::vector<c_int> A_row_indices_;     /**< row indices for constraint matrix, CSC format */
   std::vector<c_int> A_column_pointers_; /**< column pointers for constraint matrix, CSC format */

diff --git a/trajopt_sco/src/osqp_interface.cpp b/trajopt_sco/src/osqp_interface.cpp
@@ -36,7 +36,7 @@ Model::Ptr createOSQPModel(const ModelConfig::ConstPtr& config = nullptr)
   return std::make_shared<OSQPModel>(config);
 }
 
-OSQPModel::OSQPModel(const ModelConfig::ConstPtr& config) : P_(nullptr), A_(nullptr)
+OSQPModel::OSQPModel(const ModelConfig::ConstPtr& config) : P_(nullptr, free), A_(nullptr, free)
 {
   // tuning parameters to be less accurate, but add a polishing step
   if (config != nullptr)