From: Alan T. DeKok <aland@freeradius.org>
Date: Wed, 5 Feb 2025 17:54:34 +0000 (-0500)
Subject: add compile-time checks for values.
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=c2ef041905dfc6cc532e6cf293d2b19218956b25;p=thirdparty%2Ffreeradius-server.git

add compile-time checks for values.

this follows up commit 2b8351c622
---

diff --git a/src/lib/tls/conf.c b/src/lib/tls/conf.c
index 76c45b1808c..2a3f0df5f1f 100644
--- a/src/lib/tls/conf.c
+++ b/src/lib/tls/conf.c
@@ -89,7 +89,7 @@ static conf_parser_t tls_cache_config[] = {
 			 	.len = &cache_mode_table_len
 			 },
 			 .dflt = "auto" },
-	{ FR_CONF_OFFSET("name", fr_tls_cache_conf_t, id_name),
+	{ FR_CONF_OFFSET_HINT_TYPE("name", FR_TYPE_STRING, fr_tls_cache_conf_t, id_name),
 			 .dflt = "%{EAP-Type}%interpreter(server)", .quote = T_DOUBLE_QUOTED_STRING },
 	{ FR_CONF_OFFSET("lifetime", fr_tls_cache_conf_t, lifetime), .dflt = "1d" },
 
diff --git a/src/modules/rlm_attr_filter/rlm_attr_filter.c b/src/modules/rlm_attr_filter/rlm_attr_filter.c
index 11cdd0d1053..32a62e2dd5d 100644
--- a/src/modules/rlm_attr_filter/rlm_attr_filter.c
+++ b/src/modules/rlm_attr_filter/rlm_attr_filter.c
@@ -49,7 +49,7 @@ typedef struct {
 
 static const conf_parser_t module_config[] = {
 	{ FR_CONF_OFFSET_FLAGS("filename", CONF_FLAG_FILE_INPUT | CONF_FLAG_REQUIRED, rlm_attr_filter_t, filename) },
-	{ FR_CONF_OFFSET("key", rlm_attr_filter_t, key), .dflt = "&Realm", .quote = T_BARE_WORD },
+	{ FR_CONF_OFFSET_HINT_TYPE("key", FR_TYPE_STRING, rlm_attr_filter_t, key), .dflt = "&Realm", .quote = T_BARE_WORD },
 	{ FR_CONF_OFFSET("relaxed", rlm_attr_filter_t, relaxed), .dflt = "no" },
 	CONF_PARSER_TERMINATOR
 };
diff --git a/src/modules/rlm_delay/rlm_delay.c b/src/modules/rlm_delay/rlm_delay.c
index c3a48b2342b..2b2f7db3022 100644
--- a/src/modules/rlm_delay/rlm_delay.c
+++ b/src/modules/rlm_delay/rlm_delay.c
@@ -41,7 +41,7 @@ typedef struct {
  *	A mapping of configuration file names to internal variables.
  */
 static const conf_parser_t module_config[] = {
-	{ FR_CONF_OFFSET("delay", rlm_delay_t, delay) },
+	{ FR_CONF_OFFSET_HINT_TYPE("delay", FR_TYPE_TIME_DELTA, rlm_delay_t, delay) },
 	{ FR_CONF_OFFSET("relative", rlm_delay_t, relative), .dflt = "no" },
 	{ FR_CONF_OFFSET("force_reschedule", rlm_delay_t, force_reschedule), .dflt = "no" },
 	CONF_PARSER_TERMINATOR