Thanks. We will take a look and fix.


On Sat, Aug 10, 2013 at 5:10 AM, David Binderman <> 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


[msc_unicode.c:113]: (warning) Logical disjunction always evaluates to true: Code>= 0 || Code <= 65535.

Suggest replace || with &&


[libinjection/libinjection_sqli.c:777]: (warning) Logical conjunction always evaluates to false: ch < 33 && ch> 127.

Suggest replace && with ||


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.
mod-security-developers mailing list
ModSecurity Services from Trustwave's SpiderLabs: