[PATCH] (fix_led_parse_delay) value will now be set

[PATCH] (fix_led_parse_delay) value will now be set

[Florian Eckert]

---
 extensions/libxt_LED.c |    2 ++
 1 Datei geändert, 2 Zeilen hinzugefügt(+)

diff --git a/extensions/libxt_LED.c b/extensions/libxt_LED.c
index 8622c37..fd1e726 100644
--- a/extensions/libxt_LED.c
+++ b/extensions/libxt_LED.c
@@ -63,6 +63,8 @@ static void LED_parse(struct xt_option_call *cb)
 			xtables_error(PARAMETER_PROBLEM,
 				"Delay value must be within range 0..%u",
 				UINT32_MAX);
+		else
+			led->delay=atoi(cb->arg);
 		break;
 	case O_LED_ALWAYS_BLINK:
 		led->always_blink = 1;
-- 
1.7.10.4

--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] (fix_led_parse_delay) value will now be set

[Pablo Neira Ayuso]

On Tue, Aug 06, 2013 at 03:26:01PM +0200, Florian Eckert wrote:
> ---
>  extensions/libxt_LED.c |    2 ++
>  1 Datei geändert, 2 Zeilen hinzugefügt(+)
> 
> diff --git a/extensions/libxt_LED.c b/extensions/libxt_LED.c
> index 8622c37..fd1e726 100644
> --- a/extensions/libxt_LED.c
> +++ b/extensions/libxt_LED.c
> @@ -63,6 +63,8 @@ static void LED_parse(struct xt_option_call *cb)
>  			xtables_error(PARAMETER_PROBLEM,
>  				"Delay value must be within range 0..%u",
>  				UINT32_MAX);
> +		else
> +			led->delay=atoi(cb->arg);
>  		break;

We have a fix for this already:

http://git.netfilter.org/iptables/commit/?id=96c42d4c46df3edbd41fa47b860fba217f03cfeb

Will be available in upcoming iptables 1.4.20.
--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html