diff src/doveadm/dsync/dsync-slave-io.c @ 14629:c93ca5e46a8a

Marked functions parameters that are allowed to be NULL. Some APIs were also changed. The non-obvious APIs where NULL parameter was changed to "" are master_service_init() and auth_master_user_list_init(). These checks can currently be enabled only on a patched clang: http://llvm.org/bugs/show_bug.cgi?id=6786
author Timo Sirainen <tss@iki.fi>
date Sun, 24 Jun 2012 00:52:57 +0300
parents b2076acc3715
children ca37d1577291
line wrap: on
line diff
--- a/src/doveadm/dsync/dsync-slave-io.c	Fri Jun 22 21:59:01 2012 +0300
+++ b/src/doveadm/dsync/dsync-slave-io.c	Sun Jun 24 00:52:57 2012 +0300
@@ -275,7 +275,7 @@
 	slave->io = io_add(slave->fd_in, IO_READ, dsync_slave_io_input, slave);
 	o_stream_set_flush_callback(slave->output, dsync_slave_io_output, slave);
 	slave->to = timeout_add(DSYNC_SLAVE_IO_TIMEOUT_MSECS,
-				dsync_slave_io_timeout, NULL);
+				dsync_slave_io_timeout, slave);
 	o_stream_cork(slave->output);
 	o_stream_send_str(slave->output, DSYNC_HANDSHAKE_VERSION);
 
@@ -362,7 +362,7 @@
 	return 1;
 }
 
-static void ATTR_FORMAT(3, 4)
+static void ATTR_FORMAT(3, 4) ATTR_NULL(2)
 dsync_slave_input_error(struct dsync_slave_io *slave,
 			struct dsync_deserializer_decoder *decoder,
 			const char *fmt, ...)