Fixed a bug where osqpData->defines would be freed twice

osqp · Feb 20, 2024 · a7a5e89 · a7a5e89
1 parent fcc4055
commit a7a5e89
Showing 1 changed file with 1 addition and 3 deletions.
diff --git a/c_sources/osqp_mex.cpp b/c_sources/osqp_mex.cpp
@@ -145,7 +145,7 @@ void mexFunction(int nlhs, mxArray *plhs[], int nrhs, const mxArray *prhs[])
     char cmd[CMD_MAX_LEN];
 
     setString(nrhs, prhs, 0, cmd, CMD_MAX_LEN);
-
+    
     /*
      * First check to see if a new object was requested
      */
@@ -253,9 +253,7 @@ void mexFunction(int nlhs, mxArray *plhs[], int nrhs, const mxArray *prhs[])
 
     // delete the object and its data
     if (!strcmp("delete", cmd)) {
-
         osqp_cleanup(osqpData->solver);
-        if (osqpData->defines) c_free(osqpData->defines);
         destroyObject<OsqpData>(prhs[1]);
         // Warn if other commands were ignored
         if (nlhs != 0 || nrhs != 2)