[PATCH] ubox validate: port range check fix

Paul D newtwen at gmail.com
Thu Oct 17 06:37:34 PDT 2024


ping

On 2024-09-20 18:23, Paul Donald wrote:
> The luci GUI allows a single port in a port range field. This additional
> check validates a single port if a range was not found.
> 
> Signed-off-by: Paul Donald <newtwen+github at gmail.com>
> ---
>  validate/validate.c | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/validate/validate.c b/validate/validate.c
> index e72b811..ffea1d5 100644
> --- a/validate/validate.c
> +++ b/validate/validate.c
> @@ -597,8 +597,10 @@ dt_type_portrange(struct dt_state *s, int nargs)
>  
>  	n = strtoul(s->value, &e, 10);
>  
> -	if (e == s->value || *e != '-')
> -		return false;
> +	if (e == s->value || *e != '-') {
> +		// If parsing as portrange fails, try parsing as a single port
> +		return dt_type_port(s, nargs);
> +	}
>  
>  	m = strtoul(e + 1, &e, 10);
>  




More information about the openwrt-devel mailing list