[Bug 825] New: broken led-delay parameter in the LED extension

[bugzilla-daemon at netfilter.org]

https://bugzilla.netfilter.org/show_bug.cgi?id=825

           Summary: broken led-delay parameter in the LED extension
           Product: iptables
           Version: CVS (please indicate timestamp)
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: normal
          Priority: P5
         Component: iptables
        AssignedTo: netfilter-buglog at lists.netfilter.org
        ReportedBy: rom251 at yahoo.fr
   Estimated Hours: 0.0


file: extensions/libxt_LED.c [e6cf849764256651c17e9e9e3bb3037318f2b993]
https://git.netfilter.org/iptables/tree/extensions/libxt_LED.c

Hello, while trying to get the LED extension work on my build of OpenWRT I
noticed that whatever parameter of led-delay you are using (other than inf)
ends up with a rule at 0 ms.
This is due to the checked value never assigned to the structure in 
https://git.netfilter.org/iptables/tree/extensions/libxt_LED.c#n63

        else if (!xtables_strtoui(cb->arg, NULL, &delay, 0, UINT32_MAX))
            xtables_error(PARAMETER_PROBLEM,
                "Delay value must be within range 0..%u",
                UINT32_MAX);

led->delay is never assigned with the value given by the user
I changed it to

        else if (!xtables_strtoui(cb->arg, NULL, &led->delay, 0, UINT32_MAX))
            xtables_error(PARAMETER_PROBLEM,
                "Delay value must be within range 0..%u",
                UINT32_MAX);

and it is now working correctly.

-- 
Configure bugmail: https://bugzilla.netfilter.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.