From: Nick Porter <nick@portercomputing.co.uk>
Date: Thu, 22 Aug 2024 07:06:44 +0000 (+0100)
Subject: VSA and VENDOR attributes need the same encoder logic as TLV
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=e1e6d1492a28c688dfb03b41f56b81a59f1a42ef;p=thirdparty%2Ffreeradius-server.git

VSA and VENDOR attributes need the same encoder logic as TLV

Allows for %internal.encode(&Vendor-Specific) or
%internal.encode(&Vendor-Specific.Vendor) and the children will be added
correctly.
---

diff --git a/src/protocols/internal/encode.c b/src/protocols/internal/encode.c
index 1e8ef17519e..66c60ed3b80 100644
--- a/src/protocols/internal/encode.c
+++ b/src/protocols/internal/encode.c
@@ -159,9 +159,6 @@ static ssize_t internal_encode(fr_dbuff_t *dbuff,
 	 */
 	case FR_TYPE_VSA:
 	case FR_TYPE_VENDOR:
-		slen = internal_encode(&value_dbuff, da_stack, depth + 1, cursor, encode_ctx);
-		if (slen < 0) return slen;
-		break;
 
 	/*
 	 *	Children of TLVs are encoded in the context