record_accessor: release tmp_str if allocation fails

Signed-off-by: Eduardo Silva <[email protected]>
fluent · Jun 7, 2024 · 7a27bcf · 7a27bcf
1 parent 285f3dd
commit 7a27bcf
Showing 1 changed file with 2 additions and 9 deletions.
diff --git a/src/flb_record_accessor.c b/src/flb_record_accessor.c
@@ -321,18 +321,11 @@ struct flb_record_accessor *flb_ra_create(char *str, int translate_env)
         if (buf) {
             flb_sds_destroy(buf);
         }
-        return NULL;
-    }
-    ra->pattern = tmp_str;
-    if (!ra->pattern) {
-        flb_error("[record accessor] could not allocate pattern");
-        flb_free(ra);
-        if (buf) {
-            flb_sds_destroy(buf);
-        }
+        flb_sds_destroy(tmp_str);
         return NULL;
     }
 
+    ra->pattern = tmp_str;
     mk_list_init(&ra->list);
 
     /*