Home - Waterfall Grid T-Grid Console Builders Recent Builds Buildslaves Changesources - JSON API - About

Builder FreeBSD - Apache Build #10

Results:

Failed regression test

SourceStamp:

Repositoryhttps://github.com/SpiderLabs/ModSecurity
Branchnginx_refactoring
Revision33b8760e87b7441142a431175d5b459245551314
Got Revision33b8760e87b7441142a431175d5b459245551314
Changes21 changes

BuildSlave:

FreeBSD

Reason:

The AnyBranchScheduler scheduler named 'all-any' triggered this build

Steps and Logfiles:

  1. fetch update ( 12 secs )
    1. stdio
  2. autogen (ModSec) autogen ( 16 secs )
    1. stdio
  3. configure (ModSec) configure ( 3 secs )
    1. stdio
  4. make (ModSec) Compilation warnings ( 15 secs )
    1. stdio
    2. warnings (43)
  5. download downloading to check-coding_style.sh ( 1 secs )
    1. - no logs -
  6. coding style Coding style warnings ( 1 secs )
    1. stdio
    2. warnings (118)
  7. download_1 downloading to cppcheck.sh ( 1 secs )
    1. - no logs -
  8. static analisys Static analisys warnings ( 1 mins, 37 secs )
    1. stdio
    2. warnings (621)
  9. download_2 downloading to clean.sh ( 1 secs )
    1. - no logs -
  10. clean clean semaphores ( 0 secs )
    1. stdio
  11. unit test make test warnings ( 24 secs )
    1. stdio
    2. warnings (34)
  12. unit test_1 unit tests ( 19 secs )
    1. stdio
  13. clean_1 clean semaphores ( 0 secs )
    1. stdio
  14. regression test regression [8094] failed ( 4 hrs, 38 mins, 5 secs )
    1. stdio

Build Properties:

NameValueSource
branch nginx_refactoring Build
builddir /home/buildbot/buildslave/FreeBSD_-_Apache slave
buildername FreeBSD - Apache Builder
buildnumber 10 Build
codebase Build
github_repo_name ModSecurity Builder
github_repo_owner SpiderLabs Builder
got_revision 33b8760e87b7441142a431175d5b459245551314 Git
project Build
repository https://github.com/SpiderLabs/ModSecurity Build
revision 33b8760e87b7441142a431175d5b459245551314 Build
scheduler all-any Scheduler
slavename FreeBSD BuildSlave
warnings-count 816 WarningCountingShellCommand
workdir /home/buildbot/buildslave/FreeBSD_-_Apache slave (deprecated)

Forced Build Properties:

NameLabelValue

Responsible Users:

  1. Andrei Belov
  2. Felipe Zimmerle
  3. paulyang

Timing:

StartSat Aug 2 06:04:13 2014
EndSat Aug 2 10:49:41 2014
Elapsed4 hrs, 45 mins, 28 secs

All Changes:

:

  1. Change #13

    Category None
    Changed by Felipe Zimmerle <fcostaohnoyoudont@trustwave.com>
    Changed at Tue 29 Jul 2014 19:11:52
    Repository https://github.com/SpiderLabs/ModSecurity
    Branch nginx_refactoring
    Revision af65e5209c8914616ff3dcd5ccb80f37a9f94829

    Comments

    nginx refactoring
    
    Refactoring on the nginx module, including:
     - Better handling larger posts;
     - Now using nginx echo module during the regression tests.
     - Better interacting with neginx chain rules
     - Separation of the request handling and content filters.
     - Better handling nginx sessions and resource counts to allow a
       more efficient garbage collector.
     - Handling both http/1.0 and 1.1, including keep-alive.
     - Tests are now capable to test nginx as a proxy or end-server.
     - Tested agains nginx 1.6 and 1.7.

    Changed files

    • nginx/modsecurity/apr_bucket_nginx.c
    • nginx/modsecurity/ngx_http_modsecurity.c
    • tests/regression/misc/00-phases.t
    • tests/regression/misc/25-libinjection.t
    • tests/regression/nginx/conf/nginx.conf.template
  2. Change #14

    Category None
    Changed by Felipe Zimmerle <fcostaohnoyoudont@trustwave.com>
    Changed at Tue 29 Jul 2014 19:11:52
    Repository https://github.com/SpiderLabs/ModSecurity
    Branch nginx_refactoring
    Revision 67c3b61a5470984f6881af9d4931412d8ab6956c

    Comments

    nginx: looking for segfaults on the regression test.
    
    If nginx segfaults it will return, warning that the test failed.

    Changed files

    • tests/run-regression-tests-nginx.pl
  3. Change #15

    Category None
    Changed by Felipe Zimmerle <fcostaohnoyoudont@trustwave.com>
    Changed at Tue 29 Jul 2014 19:11:52
    Repository https://github.com/SpiderLabs/ModSecurity
    Branch nginx_refactoring
    Revision bb3f9d1a40d572ca5c523157b32e4e5516eb349b

    Comments

    nginx: better dealing with chunked request body

    Changed files

    • nginx/modsecurity/ngx_http_modsecurity.c
    • tests/regression/nginx/conf/nginx.conf.template
  4. Change #16

    Category None
    Changed by Felipe Zimmerle <fcostaohnoyoudont@trustwave.com>
    Changed at Tue 29 Jul 2014 19:11:52
    Repository https://github.com/SpiderLabs/ModSecurity
    Branch nginx_refactoring
    Revision 55fa1bcd985e93864998783e4e4b0baa62fe7ca7

    Comments

    nginx: copies the req body chain to be processed instead of move
    
    Add a check for the definition MOVE_REQUEST_CHAIN_TO_MODSEC, whenever it is
    set the chain will be moved into the brigade. If it was not set the chain
    will be only copied. Moving was causing segfaults on the following
    regression tests:
    
     #15 - SecRequestBodyInMemoryLimit
     #16 - SecRequestBodyInMemoryLimit (greater)
     #19 - SecRequestBodyLimitAction ProcessPartial (multipart/greater - chunked)
     (from: regression/config/10-request-directives.t)

    Changed files

    • nginx/modsecurity/apr_bucket_nginx.c
    • nginx/modsecurity/apr_bucket_nginx.h
    • nginx/modsecurity/ngx_http_modsecurity.c
  5. Change #17

    Category None
    Changed by Felipe Zimmerle <fcostaohnoyoudont@trustwave.com>
    Changed at Tue 29 Jul 2014 19:11:52
    Repository https://github.com/SpiderLabs/ModSecurity
    Branch nginx_refactoring
    Revision 6029d1b2b8f0429b0beba298c7eab04ce7c03af4

    Comments

    Regression: Accept different ports on FULL_REQUEST unit test.
    
    FULL_REQUEST unit test was expecting only for the default regression test
    suite port. Now it is expecting for [0-9]+.

    Changed files

    • tests/regression/target/00-targets.t
  6. Change #18

    Category None
    Changed by Felipe Zimmerle <fcostaohnoyoudont@trustwave.com>
    Changed at Tue 29 Jul 2014 19:11:52
    Repository https://github.com/SpiderLabs/ModSecurity
    Branch nginx_refactoring
    Revision 953b3f5b8913a3a259395383e8c350f3ac3e65dd

    Comments

    nginx: cosmetics: Splits lines longer than 80 characters

    Changed files

    • nginx/modsecurity/apr_bucket_nginx.c
    • nginx/modsecurity/apr_bucket_nginx.h
    • nginx/modsecurity/ngx_http_modsecurity.c
    • nginx/modsecurity/ngx_pool_context.c
    • nginx/modsecurity/ngx_pool_context.h
  7. Change #19

    Category None
    Changed by Felipe Zimmerle <fcostaohnoyoudont@trustwave.com>
    Changed at Tue 29 Jul 2014 19:11:53
    Repository https://github.com/SpiderLabs/ModSecurity
    Branch nginx_refactoring
    Revision a0cd48e49632d5db01f536a5179640458cdcb105

    Comments

    nginx: cosmetics: Removes trailing whitespace

    Changed files

    • nginx/modsecurity/ngx_http_modsecurity.c
    • nginx/modsecurity/ngx_pool_context.c
  8. Change #20

    Category None
    Changed by Felipe Zimmerle <fcostaohnoyoudont@trustwave.com>
    Changed at Tue 29 Jul 2014 19:11:53
    Repository https://github.com/SpiderLabs/ModSecurity
    Branch nginx_refactoring
    Revision 462440c577e4c8e0c93f28040decb0c185c66ba8

    Comments

    niginx: cosmetics: Changes CRLF to LF

    Changed files

    • nginx/modsecurity/apr_bucket_nginx.c
    • nginx/modsecurity/apr_bucket_nginx.h
  9. Change #21

    Category None
    Changed by paulyang <paulyang.infohnoyoudont@gmail.com>
    Changed at Tue 29 Jul 2014 19:11:53
    Repository https://github.com/SpiderLabs/ModSecurity
    Branch nginx_refactoring
    Revision 7af608fd34343b5a2fa640478f0d7e2cff62c817

    Comments

    Bugfix: add -P option in test script
    
    Otherwise nginx's installation directory could not be specified.
    
    Signed-off-by: paulyang <paulyang.inf@gmail.com>

    Changed files

    • tests/run-regression-tests-nginx.pl
  10. Change #22

    Category None
    Changed by Andrei Belov <defanatorohnoyoudont@gmail.com>
    Changed at Tue 29 Jul 2014 19:11:53
    Repository https://github.com/SpiderLabs/ModSecurity
    Branch nginx_refactoring
    Revision db20746d22abdbd4fc9eeea30f51cf01dbffd961

    Comments

    Removed unneeded and invalid initialization.

    Changed files

    • nginx/modsecurity/ngx_http_modsecurity.c
  11. Change #23

    Category None
    Changed by Andrei Belov <defanatorohnoyoudont@gmail.com>
    Changed at Tue 29 Jul 2014 19:11:53
    Repository https://github.com/SpiderLabs/ModSecurity
    Branch nginx_refactoring
    Revision b058a9f8ba90849262302a9e987d021bad02de84

    Comments

    Obtain port from r->connection->local_sockaddr.
    
    This eliminates segfaults caused by unset (NULL) r->port_start
    and non-NULL r->port_end. In fact, r->port_start is always NULL,
    so it is useless to rely on this pointer.

    Changed files

    • nginx/modsecurity/ngx_http_modsecurity.c
  12. Change #28

    Category None
    Changed by Felipe Zimmerle <fcostaohnoyoudont@trustwave.com>
    Changed at Wed 30 Jul 2014 21:36:46
    Repository https://github.com/SpiderLabs/ModSecurity
    Branch nginx_refactoring
    Revision 93ab6c3f72ecf5dd79fe3ab7ca3b70b8c6bfe33e

    Comments

    nginx refactoring
    
    Refactoring on the nginx module, including:
     - Better handling larger posts;
     - Now using nginx echo module during the regression tests.
     - Better interacting with neginx chain rules
     - Separation of the request handling and content filters.
     - Better handling nginx sessions and resource counts to allow a
       more efficient garbage collector.
     - Handling both http/1.0 and 1.1, including keep-alive.
     - Tests are now capable to test nginx as a proxy or end-server.
     - Tested agains nginx 1.6 and 1.7.

    Changed files

    • nginx/modsecurity/apr_bucket_nginx.c
    • nginx/modsecurity/ngx_http_modsecurity.c
    • tests/regression/misc/00-phases.t
    • tests/regression/misc/25-libinjection.t
    • tests/regression/nginx/conf/nginx.conf.template
  13. Change #29

    Category None
    Changed by Felipe Zimmerle <fcostaohnoyoudont@trustwave.com>
    Changed at Wed 30 Jul 2014 21:36:46
    Repository https://github.com/SpiderLabs/ModSecurity
    Branch nginx_refactoring
    Revision 15b5b7a2d57d90153d8c09248b96f02414b2754d

    Comments

    nginx: looking for segfaults on the regression test.
    
    If nginx segfaults it will return, warning that the test failed.

    Changed files

    • tests/run-regression-tests-nginx.pl
  14. Change #30

    Category None
    Changed by Felipe Zimmerle <fcostaohnoyoudont@trustwave.com>
    Changed at Wed 30 Jul 2014 21:36:46
    Repository https://github.com/SpiderLabs/ModSecurity
    Branch nginx_refactoring
    Revision 82e568429825d4341b57922d991c7a69eab47107

    Comments

    nginx: better dealing with chunked request body

    Changed files

    • nginx/modsecurity/ngx_http_modsecurity.c
    • tests/regression/nginx/conf/nginx.conf.template
  15. Change #31

    Category None
    Changed by Felipe Zimmerle <fcostaohnoyoudont@trustwave.com>
    Changed at Wed 30 Jul 2014 21:36:46
    Repository https://github.com/SpiderLabs/ModSecurity
    Branch nginx_refactoring
    Revision d44158fde79edb05fd98dcf0233ddc77102443c0

    Comments

    nginx: copies the req body chain to be processed instead of move
    
    Add a check for the definition MOVE_REQUEST_CHAIN_TO_MODSEC, whenever it is
    set the chain will be moved into the brigade. If it was not set the chain
    will be only copied. Moving was causing segfaults on the following
    regression tests:
    
     #15 - SecRequestBodyInMemoryLimit
     #16 - SecRequestBodyInMemoryLimit (greater)
     #19 - SecRequestBodyLimitAction ProcessPartial (multipart/greater - chunked)
     (from: regression/config/10-request-directives.t)

    Changed files

    • nginx/modsecurity/apr_bucket_nginx.c
    • nginx/modsecurity/apr_bucket_nginx.h
    • nginx/modsecurity/ngx_http_modsecurity.c
  16. Change #32

    Category None
    Changed by Felipe Zimmerle <fcostaohnoyoudont@trustwave.com>
    Changed at Wed 30 Jul 2014 21:36:46
    Repository https://github.com/SpiderLabs/ModSecurity
    Branch nginx_refactoring
    Revision 28af5c0c9dd698ea805a81b175664d5eabb1a964

    Comments

    nginx: cosmetics: Splits lines longer than 80 characters

    Changed files

    • nginx/modsecurity/apr_bucket_nginx.c
    • nginx/modsecurity/apr_bucket_nginx.h
    • nginx/modsecurity/ngx_http_modsecurity.c
    • nginx/modsecurity/ngx_pool_context.c
    • nginx/modsecurity/ngx_pool_context.h
  17. Change #33

    Category None
    Changed by Felipe Zimmerle <fcostaohnoyoudont@trustwave.com>
    Changed at Wed 30 Jul 2014 21:36:46
    Repository https://github.com/SpiderLabs/ModSecurity
    Branch nginx_refactoring
    Revision 764fa06182a8b194110a11c925dacc23d8d7ef1b

    Comments

    nginx: cosmetics: Removes trailing whitespace

    Changed files

    • nginx/modsecurity/ngx_http_modsecurity.c
    • nginx/modsecurity/ngx_pool_context.c
  18. Change #34

    Category None
    Changed by Felipe Zimmerle <fcostaohnoyoudont@trustwave.com>
    Changed at Wed 30 Jul 2014 21:36:47
    Repository https://github.com/SpiderLabs/ModSecurity
    Branch nginx_refactoring
    Revision e3583dea963ebdc93377c7ac34c72f21f1cbcb08

    Comments

    niginx: cosmetics: Changes CRLF to LF

    Changed files

    • nginx/modsecurity/apr_bucket_nginx.c
    • nginx/modsecurity/apr_bucket_nginx.h
  19. Change #35

    Category None
    Changed by paulyang <paulyang.infohnoyoudont@gmail.com>
    Changed at Wed 30 Jul 2014 21:36:47
    Repository https://github.com/SpiderLabs/ModSecurity
    Branch nginx_refactoring
    Revision 9c1cb842ea9fb890c1e6ccab854a38756a38c7a6

    Comments

    Bugfix: add -P option in test script
    
    Otherwise nginx's installation directory could not be specified.
    
    Signed-off-by: paulyang <paulyang.inf@gmail.com>

    Changed files

    • tests/run-regression-tests-nginx.pl
  20. Change #36

    Category None
    Changed by Andrei Belov <defanatorohnoyoudont@gmail.com>
    Changed at Wed 30 Jul 2014 21:36:47
    Repository https://github.com/SpiderLabs/ModSecurity
    Branch nginx_refactoring
    Revision dbc4021dfd76ed71f716152804390f0a7323596c

    Comments

    Removed unneeded and invalid initialization.

    Changed files

    • nginx/modsecurity/ngx_http_modsecurity.c
  21. Change #37

    Category None
    Changed by Andrei Belov <defanatorohnoyoudont@gmail.com>
    Changed at Wed 30 Jul 2014 21:36:47
    Repository https://github.com/SpiderLabs/ModSecurity
    Branch nginx_refactoring
    Revision 33b8760e87b7441142a431175d5b459245551314

    Comments

    Obtain port from r->connection->local_sockaddr.
    
    This eliminates segfaults caused by unset (NULL) r->port_start
    and non-NULL r->port_end. In fact, r->port_start is always NULL,
    so it is useless to rely on this pointer.

    Changed files

    • nginx/modsecurity/ngx_http_modsecurity.c