Re: [Mod-security-developers] 2.7.5 bug report

[Breno S. <bre...@gm...>]

Thanks. We will take a look and fix.

Breno


On Sat, Aug 10, 2013 at 5:10 AM, David Binderman <dc...@ho...> wrote:

> Hello there,
>
> I just ran the static analysis checker cppcheck over the 2.7.5 source code.
>
> Amongst the many things it said are
>
> 1.
>
> [msc_unicode.c:113]: (warning) Logical disjunction always evaluates to
> true: Code>= 0 || Code <= 65535.
>
> Suggest replace || with &&
>
> 2.
>
> [libinjection/libinjection_sqli.c:777]: (warning) Logical conjunction
> always evaluates to false: ch < 33 && ch> 127.
>
> Suggest replace && with ||
>
> Regards
>
> David Binderman
>
> ------------------------------------------------------------------------------
> Get 100% visibility into Java/.NET code with AppDynamics Lite!
> It's a free troubleshooting tool designed for production.
> Get down to code-level detail for bottlenecks, with <2% overhead.
> Download for free and get started troubleshooting in minutes.
> http://pubads.g.doubleclick.net/gampad/clk?id=48897031&iu=/4140/ostg.clktrk
> _______________________________________________
> mod-security-developers mailing list
> mod...@li...
> https://lists.sourceforge.net/lists/listinfo/mod-security-developers
> ModSecurity Services from Trustwave's SpiderLabs:
> https://www.trustwave.com/spiderLabs.php
>