Changeset: 18538510652d for MonetDB
URL: https://dev.monetdb.org/hg/MonetDB/rev/18538510652d
Modified Files:
        gdk/gdk_bbp.c
Branch: Dec2023
Log Message:

Free the subcommit list on failure


diffs (25 lines):

diff --git a/gdk/gdk_bbp.c b/gdk/gdk_bbp.c
--- a/gdk/gdk_bbp.c
+++ b/gdk/gdk_bbp.c
@@ -1529,11 +1529,13 @@ BBPjson_upgrade(json_storage_conversion 
                        if (!strNil(cs)) {
                                if(fixJSONStorage(&out, &cs) != GDK_SUCCEED) {
                                        TRC_CRITICAL(GDK, "could not convert 
json string for %s", cs);
+                                       GDKfree(cmlst);
                                        return GDK_FAIL;
                                }
                                if (BUNappend(newb, out, false) != GDK_SUCCEED) 
{
                                        TRC_CRITICAL(GDK, "json storage 
upgrade: appending value to bat failed");
                                        GDKfree(out);
+                                       GDKfree(cmlst);
                                        return GDK_FAIL;
                                }
                                GDKfree(out);
@@ -1542,6 +1544,7 @@ BBPjson_upgrade(json_storage_conversion 
                }
                bat_iterator_end(&bi);
                if (BBPsave(newb) != GDK_SUCCEED) {
+                       GDKfree(cmlst);
                        return GDK_FAIL;
                }
                cmlst[cnt++] = newb->batCacheid;
_______________________________________________
checkin-list mailing list -- checkin-list@monetdb.org
To unsubscribe send an email to checkin-list-le...@monetdb.org

Reply via email to