Fix/mute issues found by Coverity scan

This makes the code free of Coverity scan issues.
It is sometimes quite pedantic and expects/suggests some
coding habits, thus certain changes may look weird, but for a good
thing, I hope. The code is also tagged with Coverity scan
suppressions, to keep the code as is and hide the warning too.
Also note that Coverity treats g_return_if_fail(), g_assert() and
similar macros as unreliable, and it's true these can be disabled
during the compile time, thus it brings in other set of 'weird'
changes.

1 files changed, 2 insertions, 0 deletions

diff --git a/libemail-engine/mail-mt.c b/libemail-engine/mail-mt.c
index 19ed41b0dc..96f8e7267f 100644
--- a/libemail-engine/mail-mt.c
+++ b/libemail-engine/mail-mt.c
@@ -655,6 +655,8 @@ mail_call_main (mail_call_t type,
 	ret = m->ret;
 	mail_msg_unref (m);
 
+	/* the m->ap is freed on the message end, at do_free() above */
+	/* coverity[missing_va_end] */
 	return ret;
 }