lib-http: test-http-payload: Initialize http_context in test_client_create_clients(). Fixes complaint from scan-build.

global: don't cast NULL to void * NULL is guaranteed to be a void * thanks to dd6043c05e32a8e8db1233ed711a2c74d1477a89.

lib-http: test-http-payload: Added test with clients using shared global context.

Updated copyright notices to include the year 2018.

lib-http: test-http-payload: Terminate the test if it is hanging for some reason.

lib-http: test-http-payload: Amend shared client tests to do both download and echo tests. Also, the original download tests were named wrong.

lib-http: test-http-payload: Fixed indent problems.

lib-http: test-http-payload: Added tests for running multiple clients in parallel with shared context.

lib-http: test-http-payload: Added support for running multiple parallel clients.

global: start relying on [io]_stream_unref(NULL) being a no-op Cleanup performed with the following semantic patch: @@ expression E; @@ - if (E != NULL) { - i_stream_unref(&E); - } + i_stream_unref(&E); @@ expression E; @@ - if (E != NULL) { - o_stream_unref(&E); - } + o_stream_unref(&E);

global: start relying on io_remove{,_closed}(NULL) being a no-op Cleanup performed with the following semantic patch: @@ expression E; @@ - if (E != NULL) { - io_remove(&E); - } + io_remove(&E); @@ expression E; @@ - if (E != NULL) { - io_remove_closed(&E); - } + io_remove_closed(&E);

global: start relying on i_close_fd(-1) being a no-op Cleanup performed with the following semantic patch: @@ expression E; @@ - if (E != -1) - i_close_fd(&E); + i_close_fd(&E);

lib-http: server: Implemented API for handling the incoming request payload in the background. It allows forwarding the incoming payload to an output stream (e.g. iostream-temp) or to a buffer. The maximum size of the payload is configurable. The client will get a 413 error if the maximum is exceeded.

lib: Rename abspath to path-util

global: Update http_server_request_set_destroy_callback usage

global: Update http_client_request_set_destroy_callback usage

Updated copyright notices to include the year 2017.

global: Replaced all instances of memset(p, 0, sizeof(*p)) with the new i_zero() macro. Used the following script: C_FILES=`git ls-files *.c` H_FILES=`git ls-files *.h` for F in "$C_FILES $H_FILES"; do echo "$F" perl -p -i -e 's/safe_memset\(&\(?([^,]*)\)?,\s*0,\s*sizeof\(\g1\)\)/i_zero_safe(&$1)/g' $F perl -p -i -e 's/safe_memset\(([^,]*),\s*0,\s*sizeof\(\*\g1\)\)/i_zero_safe($1)/g' $F perl -p -i -e 's/memset\(&\(?([^,]*)\)?,\s*0,\s*sizeof\(\g1\)\)/i_zero(&$1)/g' $F perl -p -i -e 's/memset\(([^,]*),\s*0,\s*sizeof\(\*\g1\)\)/i_zero($1)/g' $F done

global: Use const for all test_functions[] arrays

lib-http: test-http-payload: Fixed problem in nested ioloop test. The running ioloop was not always stopped properly at the end of the test. This caused an assert failure in the running ioloop.

lib-http: test-http-payload: Improved logging of nested ioloop tests.

lib-http: Ensure fstream is not NULL in test Makes static analysers happier

global: Require comparisons to be strict boolean expressions * No implicit integer -> boolean or pointer -> boolean conversions * !expr can be used only if expr is boolean type These were checked with a patched clang. It found various actual bugs, which were fixed by the previous commits.

lib: remove autoclose parameter from [io]_stream_create_fd Use [io]_stream_create_fd_autoclose() for autoclose.

global: Fixed mismatched bool vs. int/pointer handling I don't think these fix any actual bugs.

o_stream_send_istream() API changed again The new API enforces the caller to correctly handle all the possible situations. It also makes in unambiguous whether to wait for input or output stream.

lib-http: test-http-payload: Added tests for the use of nested ioloops.

lib-http: test-http-payload: Added cleanup code to make sure no child processes get orphaned at failures. Also handles SIGSEGV and SIGABRT.

lib-http: test-http-payload: Added -D option to easily enable debug mode.

global: Simplify code with the new o_stream_send_istream() API

global: Don't assume o_stream_send_istream() return bytes sent. This won't be true soon after its API changes.

global: Replaced all i_stream_read_data() with _more() and _bytes()

lib-http: client: Improved test-http-payload to test partial reading of response payload.

lib-http: server: Improved test-http-payload to test partial reading of request payload.

lib-http: Created test program that tests payload exchange between client and server with various configurations. It recursively uses all files in the current directory as payload. It is currenrly not part of `make check', because it is a stress-testing tool that can run for a long time.