From: Alan T. DeKok <aland@freeradius.org>
Date: Thu, 7 Oct 2021 21:56:29 +0000 (-0400)
Subject: don't abort when decode_value() fails.
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=c8a62ec1417f56d94d882c871c0645286edc9b2f;p=thirdparty%2Ffreeradius-server.git

don't abort when decode_value() fails.

Instead, just treat the entire struct as being raw
---

diff --git a/src/lib/util/struct.c b/src/lib/util/struct.c
index b74876aa9e7..96ad46ba2a5 100644
--- a/src/lib/util/struct.c
+++ b/src/lib/util/struct.c
@@ -264,7 +264,7 @@ ssize_t fr_struct_from_network(TALLOC_CTX *ctx, fr_dcursor_t *cursor,
 			slen = decode_value(child_ctx, &child_cursor, fr_dict_by_da(child), child, p, child_length, decode_ctx);
 			if (slen < 0) {
 				FR_PROTO_TRACE("Failed decoding value");
-				return slen - (p - data);
+				goto unknown;
 			}
 
 			p += slen;   	/* not always the same as child->flags.length */