commit b2b51369d023d17d6703b70df8bd4f9a3fb7825f
author aPiecek <piecek@cesnet.cz> Thu Jun 22 09:14:59 2023 +0200
committer Michal Vasko <mvasko@cesnet.cz> Tue Jul 04 14:31:45 2023 +0200
tree 5f2e2beeadc4ef9cf7c35123c63e946056fae963
parent ae7eae111e4ba77d7cd8bc4e67ee671f4eafc616

yangre BUGFIX in add_pattern()

Memory problems in case of error.

tools/lint/yl_schema_features.c

diff --git a/tools/lint/yl_schema_features.c b/tools/lint/yl_schema_features.c
index b452496..ceaff36 100644
--- a/tools/lint/yl_schema_features.c
+++ b/tools/lint/yl_schema_features.c
@@ -63,26 +63,21 @@
 int
 parse_features(const char *fstring, struct ly_set *fset)
 {
-    struct yl_schema_features *rec;
+    struct yl_schema_features *rec = NULL;
     uint32_t count;
     char *p, **fp;
 
     rec = calloc(1, sizeof *rec);
     if (!rec) {
         YLMSG_E("Unable to allocate features information record (%s).\n", strerror(errno));
-        return -1;
-    }
-    if (ly_set_add(fset, rec, 1, NULL)) {
-        YLMSG_E("Unable to store features information (%s).\n", strerror(errno));
-        free(rec);
-        return -1;
+        goto error;
     }
 
     /* fill the record */
     p = strchr(fstring, ':');
     if (!p) {
         YLMSG_E("Invalid format of the features specification (%s).\n", fstring);
-        return -1;
+        goto error;
     }
     rec->mod_name = strndup(fstring, p - fstring);
 
@@ -103,7 +98,7 @@
             fp = realloc(rec->features, (count + 1) * sizeof *rec->features);
             if (!fp) {
                 YLMSG_E("Unable to store features list information (%s).\n", strerror(errno));
-                return -1;
+                goto error;
             }
             rec->features = fp;
             fp = &rec->features[count++]; /* array item to set */
@@ -115,12 +110,23 @@
     fp = realloc(rec->features, (count + 1) * sizeof *rec->features);
     if (!fp) {
         YLMSG_E("Unable to store features list information (%s).\n", strerror(errno));
-        return -1;
+        goto error;
     }
     rec->features = fp;
     rec->features[count++] = NULL;
 
+    /* Store record to the output set. */
+    if (ly_set_add(fset, rec, 1, NULL)) {
+        YLMSG_E("Unable to store features information (%s).\n", strerror(errno));
+        goto error;
+    }
+    rec = NULL;
+
     return 0;
+
+error:
+    yl_schema_features_free(rec);
+    return -1;
 }
 
 void