Remove initial FR_TYPE_NULL check from fr_value_box_from_substr() (CID #1504016)...

Better to let it be handled in the switch statement.

diff --git a/src/lib/util/value.c b/src/lib/util/value.c
index 4c469f1bffb26bf3c9962005aff4cad4661a3542..ab51b8c9e165b7d479f875d10a749d9987c6a2e5 100644 (file)
--- a/src/lib/util/value.c
+++ b/src/lib/util/value.c
@@ -4581,8 +4581,6 @@ ssize_t fr_value_box_from_substr(TALLOC_CTX *ctx, fr_value_box_t *dst,
        char                            buffer[256];
        fr_slen_t                       slen;
 
-       if (!fr_cond_assert(dst_type != FR_TYPE_NULL)) return -1;
-
        if (!rules) rules = &default_rules;
 
        fr_strerror_clear();