From 9d841382169f90f542120eade1f10e5d5d8baecc Mon Sep 17 00:00:00 2001 From: Holger Vogt Date: Sat, 9 Sep 2023 23:35:54 +0200 Subject: [PATCH] Remove memory leak by not mallocing unused node_ids. Remove some compiler warnings. --- src/osdi/osdisetup.c | 16 ++++++++++------ 1 file changed, 10 insertions(+), 6 deletions(-) diff --git a/src/osdi/osdisetup.c b/src/osdi/osdisetup.c index 08bae9942..f1a868761 100644 --- a/src/osdi/osdisetup.c +++ b/src/osdi/osdisetup.c @@ -478,8 +478,10 @@ int OSDIbindCSC(GENmodel *inModel, CKTcircuit *ckt) { GENmodel *gen_model; GENinstance *gen_inst; - /* setup a temporary buffer */ - uint32_t *node_ids = TMALLOC(uint32_t, descr->num_nodes); + NG_IGNORE(ckt); + + /* setup a temporary buffer + uint32_t *node_ids = TMALLOC(uint32_t, descr->num_nodes);*/ for (gen_model = inModel; gen_model; gen_model = gen_model->GENnextModel) { void *model = osdi_model_data(gen_model); @@ -503,8 +505,10 @@ int OSDIupdateCSC(GENmodel *inModel, CKTcircuit *ckt, bool complex) { GENmodel *gen_model; GENinstance *gen_inst; - /* setup a temporary buffer */ - uint32_t *node_ids = TMALLOC(uint32_t, descr->num_nodes); + NG_IGNORE(ckt); + + /* setup a temporary buffer + uint32_t *node_ids = TMALLOC(uint32_t, descr->num_nodes);*/ for (gen_model = inModel; gen_model; gen_model = gen_model->GENnextModel) { void *model = osdi_model_data(gen_model); @@ -522,10 +526,10 @@ int OSDIupdateCSC(GENmodel *inModel, CKTcircuit *ckt, bool complex) { return (OK); } int OSDIbindCSCComplexToReal(GENmodel *inModel, CKTcircuit *ckt) { - OSDIupdateCSC(inModel, ckt, false); + return OSDIupdateCSC(inModel, ckt, false); } int OSDIbindCSCComplex(GENmodel *inModel, CKTcircuit *ckt) { - OSDIupdateCSC(inModel, ckt, true); + return OSDIupdateCSC(inModel, ckt, true); } #endif