rlm_sql variable substitution clarification

[Alan DeKok]

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.
 			 */

  Alan DeKok.