From: Alan T. DeKok <aland@freeradius.org>
Date: Fri, 6 Sep 2024 14:17:12 +0000 (-0400)
Subject: return code is signed
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=965161d7c72dd9477647607fa76274751e75bc89;p=thirdparty%2Ffreeradius-server.git

return code is signed
---

diff --git a/src/lib/unlang/compile.c b/src/lib/unlang/compile.c
index c3f6c0306a6..5bd559f077c 100644
--- a/src/lib/unlang/compile.c
+++ b/src/lib/unlang/compile.c
@@ -4006,7 +4006,7 @@ static unlang_t *compile_subrequest(unlang_t *parent, unlang_compile_t *unlang_c
 	 */
 	if ((name2[0] == '&') ||
 	    (tmpl_require_enum_prefix && ((p = strchr(name2, ':')) == NULL))) {
-		size_t slen;
+		ssize_t slen;
 
 		slen = tmpl_afrom_attr_substr(parent, NULL, &vpt,
 					      &FR_SBUFF_IN(name2, talloc_array_length(name2) - 1),
@@ -4016,6 +4016,8 @@ static unlang_t *compile_subrequest(unlang_t *parent, unlang_compile_t *unlang_c
 			return NULL;
 		}
 
+		fr_assert(tmpl_is_attr(vpt));
+
 		/*
 		 *	Anything resembling an integer or string is
 		 *	OK.  Nothing else makes sense.