fix error messages

FreeRADIUS · Dec 6, 2019 · 400678a · 400678a
1 parent df02bbc
commit 400678a
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 8 deletions.
diff --git a/src/lib/util/dict_tokenize.c b/src/lib/util/dict_tokenize.c
@@ -390,11 +390,6 @@ static int dict_process_flag_field(dict_tokenize_ctx_t *ctx, char *name, fr_type
 				fr_strerror_printf("The 'length' flag can only be used with value 'uint16'");
 			}
 
-			if ((type != FR_TYPE_STRING) && (type != FR_TYPE_OCTETS) && (type != FR_TYPE_STRUCT)) {
-				fr_strerror_printf("The 'length' flag can only be used for attributes of type 'string' or 'octets'");
-				return -1;
-			}
-
 			flags->extra = 1;
 			flags->subtype = FLAG_LENGTH_UINT16;
 

diff --git a/src/lib/util/dict_validate.c b/src/lib/util/dict_validate.c
@@ -267,9 +267,10 @@ bool dict_attr_flags_valid(fr_dict_t *dict, fr_dict_attr_t const *parent,
 
 		if ((flags->subtype == FLAG_LENGTH_UINT16) &&
 		    ((type != FR_TYPE_STRING) && (type != FR_TYPE_OCTETS) && (type != FR_TYPE_STRUCT))) {
-				fr_strerror_printf("The 'length' flag can only be used for attributes of type 'string' or 'octets'");
-				return -1;
-			}
+			fr_strerror_printf("The 'length' flag cannot be used used with type %s",
+					   fr_table_str_by_value(fr_value_box_type_table, type, "<UNKNOWN>"));
+			return -1;
+		}
 
 		FORBID_OTHER_FLAGS(extra);
 	}