From: Alan T. DeKok <aland@freeradius.org>
Date: Wed, 9 Feb 2011 10:53:50 +0000 (+0100)
Subject: Allow spaces and CRs
X-Git-Tag: release_2_1_11~132
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=77f7b60726871cd8fc1a1f07746b14d722b33419;p=thirdparty%2Ffreeradius-server.git

Allow spaces and CRs

Now that the underlying redis escape function handles spaces and
control characters
---

diff --git a/src/modules/rlm_rediswho/rlm_rediswho.c b/src/modules/rlm_rediswho/rlm_rediswho.c
index 0755094506d..5059f9d8597 100644
--- a/src/modules/rlm_rediswho/rlm_rediswho.c
+++ b/src/modules/rlm_rediswho/rlm_rediswho.c
@@ -133,7 +133,7 @@ static int rediswho_command(const char *fmt, REDISSOCK *dissocket,
 		break;
 	default:
 		break;
-	};
+	}
 
 	(data->redis_inst->redis_finish_query)(dissocket);
 
@@ -343,16 +343,6 @@ static int rediswho_accounting(void * instance, REQUEST * request)
 		return RLM_MODULE_NOOP;
 	}
 
-	if (strchr(request->username->vp_strvalue, ' ')) {
-		RDEBUG("Spaces are not allowed in the User-Name");
-		return RLM_MODULE_NOOP;
-	}
-
-	if (strchr(request->username->vp_strvalue, '\n')) {
-		RDEBUG("CR are not allowed in the User-Name");
-		return RLM_MODULE_NOOP;
-	}
-
 	dissocket = data->redis_inst->redis_get_socket(data->redis_inst);
 	if (dissocket == NULL) {
 		RDEBUG("cannot allocate redis connection");