From ec0a604f274b28ff397c6490b4556519520e99a6 Mon Sep 17 00:00:00 2001 From: Dragan Dosen Date: Mon, 22 Feb 2021 17:39:02 +0100 Subject: [PATCH] CLEANUP: vars: make smp_fetch_var() to reuse vars_get_by_desc() They both do the same thing, so let's remove unneeded code duplication. --- src/vars.c | 25 +------------------------ 1 file changed, 1 insertion(+), 24 deletions(-) diff --git a/src/vars.c b/src/vars.c index 251ffe121..3da443002 100644 --- a/src/vars.c +++ b/src/vars.c @@ -332,31 +332,8 @@ static inline struct var *var_get(struct vars *vars, const char *name) static int smp_fetch_var(const struct arg *args, struct sample *smp, const char *kw, void *private) { const struct var_desc *var_desc = &args[0].data.var; - struct var *var; - struct vars *vars; - /* Check the availibity of the variable. */ - vars = get_vars(smp->sess, smp->strm, var_desc->scope); - if (!vars || vars->scope != var_desc->scope) - return 0; - - HA_RWLOCK_RDLOCK(VARS_LOCK, &vars->rwlock); - var = var_get(vars, var_desc->name); - - /* check for the variable avalaibility */ - if (!var) { - HA_RWLOCK_RDUNLOCK(VARS_LOCK, &vars->rwlock); - return 0; - } - - /* Duplicate the sample data because it could modified by another - * thread */ - smp->data = var->data; - smp_dup(smp); - smp->flags |= SMP_F_CONST; - - HA_RWLOCK_RDUNLOCK(VARS_LOCK, &vars->rwlock); - return 1; + return vars_get_by_desc(var_desc, smp); } /* This function search in the a variable with the same