enygma / expose

An Intrusion Detection System library loosely based on PHP IDS
MIT License
266 stars 55 forks source link

Fixed bug where for-loop variable name was reused. #20

Closed gheld closed 11 years ago

gheld commented 11 years ago

There are two loops, in the outer one $index is the name of the variable being examined and in the inner on $index was the number of the filter. This caused the wrong value to be passed to the Report constructor.

Suggesting alternate naming for the inner loop.

enygma commented 11 years ago

Thanks for catching that and for the pull request! :)