[Pkg-citadel-devel] Bug#749378: citadel: Excess arguments in call to smtp_try_one_queue_entry

Michael Tautschnig mt at debian.org
Mon May 26 16:53:40 UTC 2014


Package: citadel
Version: 8.24-1
Severity: minor
Usertags: goto-cc

During a rebuild of all Debian packages in a clean sid chroot (using cowbuilder
and pbuilder) the build failed with the following error. Please note that we
use our research compiler tool-chain (using tools from the cbmc package), which
permits extended reporting on type inconsistencies at link time.

[...]
CC md5.c

error: conflicting function declarations "smtp_try_one_queue_entry"
old definition in module serv_smtpeventclient file modules/smtp/serv_smtpeventclient.c line 619
void (struct queueitem *MyQItem, struct _mailq_entry *MyQEntry, struct StrBuf *MsgText, signed int KeepMsgText, signed int MsgCount)
new definition in module serv_smtpqueue file modules/smtp/serv_smtpqueue.c line 115
void (struct queueitem *, struct _mailq_entry *, struct StrBuf *, signed int, signed int, struct ParsedURL *)
Makefile:142: recipe for target 'citserver' failed
make[1]: *** [citserver] Error 64

The misleading declaration may be reviewed here:

http://sources.debian.net/src/citadel/8.24-1/modules/smtp/serv_smtpqueue.c?hl=115#L115

Observe the extra argument RelayUrls (which is also passed along in calls),
compared to the actual definition:

http://sources.debian.net/src/citadel/8.24-1/modules/smtp/serv_smtpeventclient.c?hl=619#L619

This may hint at an incomplete refactoring and possibly other errors.

Best,
Michael

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 859 bytes
Desc: not available
URL: <http://lists.alioth.debian.org/pipermail/pkg-citadel-devel/attachments/20140526/abd80c6c/attachment.sig>


More information about the Pkg-citadel-devel mailing list