Bug#347657: please add imapd.patch from kolab upstream

[Sven Mueller]

Ondrej Sury wrote on 12/01/2006 20:25:
>>>-static void db_err(const char *db_prfx, char *buffer)
>>>+static void db_err(const DB_ENV *dbenv, const char *db_prfx, const char *buffer)
>>> {
>>>     syslog(LOG_WARNING, "DBERROR %s: %s", db_prfx, buffer);
>>> }
>>
>>We should verify this one.
> 
> 
> AFAIK I remember something like this from info-cyrus list to be related
> to db4.4.  But I could be wrong...

I honestly don't understand wether that patch is necessary or not. In
db.h from libdb4.2-dev, there are two definitions for the set_errcall()
function (which sets the callback for db errors, used in
lib/cyrusdb_berkeley.c):

in struct __db:
 void (*set_errcall) __P((DB *, void (*)(const char *, char *)));
in struct __db_env:
 void (*set_errcall) __P((DB_ENV *, void (*)(const char *, char *)));

Which leads me to _think_ we need the patch, since it is called as:
 dbenv->set_errcall(dbenv, db_err);

dbenv is of type DB_ENV*, which is actually (struct __db_env)*.

Should I apply the patch? I think so, but I'm not really sure.

cu,
sven