stats: Populate group_by field earlier in settings check

This is needed in next commit to avoid crash.

diff --git a/src/stats/stats-settings.c b/src/stats/stats-settings.c
index 4ca9abfe377506d78b25ca9042b2b9bd576dad50..78402d078092a99b7825650cf012727a29994d88 100644 (file)
--- a/src/stats/stats-settings.c
+++ b/src/stats/stats-settings.c
@@ -459,6 +459,7 @@ static bool parse_metric_group_by(struct stats_metric_settings *set,
 
                /* <field name>:<aggregation func>... */
                params = t_strsplit(*tmp, ":");
+               group_by.field = p_strdup(pool, params[0]);
 
                if (params[1] == NULL) {
                        /* <field name> - alias for <field>:discrete */
@@ -485,8 +486,6 @@ static bool parse_metric_group_by(struct stats_metric_settings *set,
                        return FALSE;
                }
 
-               group_by.field = p_strdup(pool, params[0]);
-
                array_push_back(&set->parsed_group_by, &group_by);
        }