From: Arran Cudbard-Bell <a.cudbardb@freeradius.org>
Date: Wed, 23 Oct 2024 05:00:11 +0000 (-0600)
Subject: internal encoder: ACTUALLY skip name only attributes, instead of just infinitely... 
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=e507b0f048545a649955f4f436c3ffe8751e2efe;p=thirdparty%2Ffreeradius-server.git

internal encoder: ACTUALLY skip name only attributes, instead of just infinitely looping
---

diff --git a/src/protocols/internal/encode.c b/src/protocols/internal/encode.c
index 66c60ed3b80..2a61464a5cd 100644
--- a/src/protocols/internal/encode.c
+++ b/src/protocols/internal/encode.c
@@ -73,7 +73,10 @@ static ssize_t internal_encode(fr_dbuff_t *dbuff,
 	 *	Silently skip name only attributes if we're writing
 	 *	to a database or cache.
 	 */
-	if (!our_encode_ctx->allow_name_only && vp->da->flags.name_only) return 0;
+	if (!our_encode_ctx->allow_name_only && vp->da->flags.name_only) {
+		fr_dcursor_next(cursor);
+		return 0;
+	}
 
 	FR_PROTO_STACK_PRINT(da_stack, depth);