Check for NULL, and free on error. CID #1503946

diff --git a/src/modules/rlm_csv/rlm_csv.c b/src/modules/rlm_csv/rlm_csv.c
index 37453281eee6b4373856aa22c764736f066dfea3..3fb4ac47c12ceb5207d3875029a3b0cf15f2dc09 100644 (file)
--- a/src/modules/rlm_csv/rlm_csv.c
+++ b/src/modules/rlm_csv/rlm_csv.c
@@ -230,9 +230,13 @@ static bool duplicate_entry(CONF_SECTION *conf, rlm_csv_t *inst, rlm_csv_entry_t
        talloc_set_type(e, rlm_csv_entry_t);
 
        e->key = fr_value_box_alloc_null(e);
+       if (!e->key) goto fail;
+
        if (fr_value_box_from_str(e->key, e->key, type, NULL, p, strlen(p), NULL, false) < 0) {
                cf_log_err(conf, "Failed parsing key field in file %s line %d - %s", inst->filename, lineno,
                           fr_strerror());
+       fail:
+               talloc_free(e);
                return false;
        }