commit 901e31e: Revert "[Fix] #3129 Multiple classifiers on redis working incorrectly"

[resec]

Author: resec
Date: 2019-11-14 16:04:24 +0800
URL: https://github.com/rspamd/rspamd/commit/901e31e9772fcdf1043ace5536805c3c5e5b90c3 (refs/pull/3147/head)

Revert "[Fix] #3129 Multiple classifiers on redis working incorrectly"
This reverts commit 14495547dea7e4a0c14cf1507ba7def74b58a4ce.

---
 src/libstat/backends/redis_backend.c  | 20 ++------------------
 src/libstat/learn_cache/redis_cache.c | 12 +-----------
 2 files changed, 3 insertions(+), 29 deletions(-)

diff --git a/src/libstat/backends/redis_backend.c b/src/libstat/backends/redis_backend.c
index 70011a628..08edf1a4f 100644
--- a/src/libstat/backends/redis_backend.c
+++ b/src/libstat/backends/redis_backend.c
@@ -1425,29 +1425,13 @@ rspamd_redis_parse_classifier_opts (struct redis_stat_ctx *backend,
 
 	elt = ucl_object_lookup (obj, "prefix");
 	if (elt == NULL || ucl_object_type (elt) != UCL_STRING) {
-		gchar *redis_object;
-
 		/* Default non-users statistics */
 		if (backend->enable_users || backend->cbref_user != -1) {
-			redis_object = REDIS_DEFAULT_USERS_OBJECT;
+			backend->redis_object = REDIS_DEFAULT_USERS_OBJECT;
 		}
 		else {
-			redis_object = REDIS_DEFAULT_OBJECT;
-		}
-
-		/* Prepend classifier name if defined */
-		elt = ucl_object_lookup (obj, "name");
-		if (elt != NULL && ucl_object_type (elt) == UCL_STRING) {
-			const gchar *cl_name = ucl_object_tostring (elt);
-			gchar *temp;
-			temp = g_malloc (strlen (cl_name) + strlen (redis_object) + 2);
-			strcpy (temp, cl_name);
-			strcat (temp, "_");
-			strcat (temp, redis_object);
-			redis_object = temp;
+			backend->redis_object = REDIS_DEFAULT_OBJECT;
 		}
-
-		backend->redis_object = redis_object;
 	}
 	else {
 		/* XXX: sanity check */
diff --git a/src/libstat/learn_cache/redis_cache.c b/src/libstat/learn_cache/redis_cache.c
index 320f422dd..0df3783ab 100644
--- a/src/libstat/learn_cache/redis_cache.c
+++ b/src/libstat/learn_cache/redis_cache.c
@@ -292,17 +292,7 @@ rspamd_stat_cache_redis_init (struct rspamd_stat_ctx *ctx,
 		cache_ctx->redis_object = ucl_object_tostring (obj);
 	}
 	else {
-		gchar *cl_name = st->classifier->cfg->name;
-		if (cl_name) {
-			gchar *redis_object;
-			redis_object = g_malloc (strlen (cl_name) + strlen (DEFAULT_REDIS_KEY) + 2);
-			strcpy (redis_object, cl_name);
-			strcat (redis_object, "_");
-			strcat (redis_object, DEFAULT_REDIS_KEY);
-			cache_ctx->redis_object = redis_object;
-		} else {
-			cache_ctx->redis_object = DEFAULT_REDIS_KEY;
-		}
+		cache_ctx->redis_object = DEFAULT_REDIS_KEY;
 	}
 
 	cache_ctx->conf_ref = conf_ref;