Thanks. We will take a look and fix.

Breno


On Sat, Aug 10, 2013 at 5:10 AM, David Binderman <dcb314@hotmail.com> 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-security-developers@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mod-security-developers
ModSecurity Services from Trustwave's SpiderLabs:
https://www.trustwave.com/spiderLabs.php