rlm_sql variable substitution clarification

[Guy Fraser]

On Fri, 2006-07-07 at 14:18 -0400, Alan DeKok wrote:
> Guy Fraser <guy at incentre.net> wrote:
> > The attributes are not named like they were in Cistron dictionaries.
> > They all start with "X-".
> 
>   There's still a bug:
> 
>   Reply-Message = `%{Reply-Message:-x%{User-Password}x}`
> 
>   returns "xbob" for the standard test of user "bob/bob".
> 
>   Patch is given below.
> 
> Index: src/main/xlat.c
> ===================================================================
> RCS file: /source/radiusd/src/main/xlat.c,v
> retrieving revision 1.72.2.7.2.1
> diff -u -r1.72.2.7.2.1 xlat.c
> --- src/main/xlat.c	8 Dec 2005 12:47:56 -0000	1.72.2.7.2.1
> +++ src/main/xlat.c	7 Jul 2006 18:24:08 -0000
> @@ -533,7 +533,7 @@
>  	 * useless if we found what we need
>  	 */
>  	if (found) {
> -		while((*p != '\0') && (openbraces > 0)) {
> +		while((*p != '\0') && (openbraces > *open)) {
>  			/*
>  			 *	Handle escapes outside of the loop.
>  			 */
> 
Thank you, I'll give it a shot.