return underlying error rather than over-writing it

diff --git a/src/protocols/radius/encode.c b/src/protocols/radius/encode.c
index fb6af876afdeac103a6b5feee82abbea0aa4c4dc..1cf5e464993c0651f479d52ea51af021ec0789aa 100644 (file)
--- a/src/protocols/radius/encode.c
+++ b/src/protocols/radius/encode.c
@@ -565,10 +565,7 @@ static ssize_t encode_value(fr_dbuff_t *dbuff,
                }
 
                slen = encode_tunnel_password(&work_dbuff, &value_start, fr_dbuff_used(&value_dbuff), packet_ctx);
-               if (slen < 0) {
-                       fr_strerror_printf("%s too long", vp->da->name);
-                       return slen;
-               }
+               if (slen < 0) return slen;
 
                encrypted = true;
                break;

diff --git a/src/tests/unit/protocols/radius/tunnel.txt b/src/tests/unit/protocols/radius/tunnel.txt
index 9f4b574ddb11f3ef30a47f531dacfdafad00fde4..e996bb9279dbb016cf6e5b8ca1745705f20623e8 100644 (file)
--- a/src/tests/unit/protocols/radius/tunnel.txt
+++ b/src/tests/unit/protocols/radius/tunnel.txt
@@ -96,7 +96,7 @@ match Tunnel-Password = "xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
 #  250 octets is not OK.
 #
 encode-pair Tunnel-Password = "xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx123456789a"
-match Tunnel-Password too long
+match Input password is too large for tunnel password encoding
 
 returned
 match -253