ostream-zlib: Ignore missing finish if parent stream is ignoring errors This fixes panic with imap_zlib plugin when client enables the IMAP COMPRESS extension and disconnects: Panic: file ostream-zlib.c: line 36 (o_stream_zlib_close): assertion failed: (zstream->ostream.finished || zstream->ostream.ostream.stream_errno != 0)

Updated copyright notices to include the year 2018.

global: Use o_stream_flush_parent()

lib-compression: ostream-zlib - Require o_stream_finish() to write the trailer It's now possible to call o_stream_flush() without the trailer getting written. It's also now required to call o_stream_finish() to write the trailer before closing the ostream. This guarantees that the caller checks that the trailer was successfully written. This isn't needed if the ostream already failed before writing the trailer.

Updated copyright notices to include the year 2017.

global: Change string position/length from unsigned int to size_t Mainly to avoid truncating >4GB strings, which might potentially cause some security holes. Normally there are other limits, which prevent such excessive strings from being created in the first place. I'm sure this didn't find everything. Maybe everything could be found with compiler warnings. -Wconversion kind of does it, but it gives way too many unnecessary warnings. These were mainly found with: grep " = strlen" egrep "unsigned int.*(size|len)"

global: unsigned int:1 -> bool:1 perl -i -pe 's/unsigned int ([^,:;]+):1;/bool $1:1;/' **/*.[ch]

lib-compression: Changed impossible errors to asserts. o_stream_flush() no longer calls the stream's flush() if stream has failed.

global: freshen copyright git ls-files | xargs perl -p -i -e 's/(\d+)-201[0-5]/$1-2016/g;s/ (201[0-5]) Dovecot/ $1-2016 Dovecot/'

global: freshen copyright Robomatically: git ls-files | xargs perl -p -i -e 's/(\d+)-201[0-4]/$1-2015/g;s/ (201[0-4]) Dovecot/ $1-2015 Dovecot/' Happy 2015 everyone! Signed-off-by: Phil Carmody <phil@dovecot.fi>

lib-compression: Compression ostreams may have caused parent ostream to use too much memory.

lib-compression: gz compression didn't handle multiple flushes correctly.

Updated copyright notices to include year 2014.

iostreams: Added close_parent flag to close() handler and clarified close/destroy APIs. This makes it unambiguous how things work: Unless you explicitly call [io]_stream_close(), the parent streams won't be closed. This is what most (hopefully all!) of the existing code expects. I was wondering a bit if [io]_stream_destroy() should simply have been removed and replaced with [io]_stream_unref() calls, since they would have worked basically everywhere, but there might be some places where it's better to have explicitly closed the stream (and where closing the parent stream doesn't matter).

Oops :) Update copyrights to 2013 without breaking all .c files.

Updated copyright notices to include year 2013.

Added o_stream_get_fd().

Moved zlib/bzlib code to lib-compression library.