rlm_rediswho with spaces

Alan DeKok aland at deployingradius.com
Fri Oct 26 13:49:34 CEST 2012


Brian Candler wrote:
> So that's the first problem to fix.  In rlm_rediswho.c I suspect this
> line...
> 
>                 if (!radius_xlat(query, sizeof (query), fmt, request, NULL)) {
> 
> ...just needs the NULL changing to inst->redis_escape_func.

  Fixed && pushed.

> I think what's needed is an inverse of redis_escape when building the
> command and args, which should be pretty straightforward.

  I'm not sure that's a good idea.

  Alan DeKok.


More information about the Freeradius-Devel mailing list