Garbage after the packet is a warning, not an error

diff --git a/src/modules/proto_dhcp/dhcp.c b/src/modules/proto_dhcp/dhcp.c
index d8c03d5d7f663bcec086d87460fc47c2c120af34..1ba568322c33dcd527f3016545edc43d9f7e3218 100644 (file)
--- a/src/modules/proto_dhcp/dhcp.c
+++ b/src/modules/proto_dhcp/dhcp.c
@@ -1071,9 +1071,8 @@ ssize_t fr_dhcp_decode_option(TALLOC_CTX *ctx, vp_cursor_t *cursor,
 
                for (i = 1; i < data_len; i++) {
                        if (p[i] != 0) {
-                               fr_strerror_printf("%s: Non padding option follows end of options signifier",
-                                                  __FUNCTION__);
-                               return -1;
+                               WARN("DHCP: Non padding option follows end of options signifier");
+                               break;
                        }
                }
                return data_len;