changing internal version number to 8.12

NetQueue: make logging runtime configurable; add context.

NSYN: fix choosing of remote party.

MODULES: add priorities in execution

  - specialy the EV_TIMER callbacks were suffering from wrong order execution which caused bad behaviours:
    - network queueing now is executed before mail queue handling -> mailinglist posts are sent in < 60s instead of < 120s as before
    - network spool consolidation is done before running the citadel networkers; that way the NTT-list won't be locked by running networkers

NetworkClient: fix filedescriptor leaks

  - in all cases where we're done with the spoolfile we need to close its handle & flush the FDIO buffer if already alloc'd

EVENT Client: parametrize when to close the fd on detach or not

  - we shouldn't close our FD if we want to continue using it later on after doing i.e. database io.

POP3Client: fix logging

  - add final statistic statement with count
  - remove uber-verbose log-statements
  - make the rest of the logging configurable.

EVENT-Client: don't call the timeout function directly

  - if we call the timeout function directly we might find upper stackframes which rely on the event-context still being valid. Thus we rather set a timeout of 0.0 which then calls the timeout function and kills the context.

EVENTClient: rather use the FD found on our internal structures then searching it on the watcher struct; its not allways there.

Merge branch 'master' of ssh://git.citadel.org/appl/gitroot/citadel

serv_read_binary(): abort if the server status returns anything else then 6xx

cmd_read(): fseek just returns 0 on success, not the current offset.

remove duplicate definition in it.po

NSYN: adjust to new semantics

  - load stuff we need to properly decide whether to spool files or not.

Networkespool: use FileMoveChunked(); since the others won't read or won't write with offset.

Add FileMoveChunked(); which can read and write with offsets.

EVENT: when handling errors EAGAIN isn't fatal.

CURL: just call init once.

Merge branch 'master' of ssh://git.citadel.org/appl/gitroot/citadel

SMTP-Client: fix target port handling for relay & fallback hosts

Use right size value as input for splice.

Networkclient: opend spoolfile with proper flags for splice(); fix some minor stability issues

FileSendChunked(): replace sendfile by splice()
FileSendChunked+FileRecvChunked: handle situations in which the first splice reads more then the second can send.

NETP: extract password string when we need it. else it might be overwritten meanwhile.

networker: use the right char buffer for the password.

Networker: Follow changes of networking infrastructure in the networker client

Networker: also register STOP hook, so if sessions are timed out or killed the NTT-List is cleaned up.

Rework networker

  - keep netconfigs in hash; create the usual deserialize/delete/... functions
  - keep the netmap in hash; create the usual de/serialize/delete/... functions
  - change sequence: serialize & save netmap; combine spoolfiles; free netmap.
  - use strbufs splice() wrappers to combine the message-spoolfiles into host-spoolfiles instead of forking a shell & running cat + rm
  - when appending to host-spoolfile fails, remove partial adds to avoid creating invalid spool control files
  - only cleanup spool directory if we successfully combined all files; else we might loose messages if i.e. a networker is running for that node.

StrBufExtract_tokenFromStr: add wrapper around StrBufExtract_token() which takes a char* as string to de-tokenize.

READ: sanitize values & output errors on invalid parameters.

WRIT: catch silly send size values.

Merge branch 'master' of ssh://git.citadel.org/appl/gitroot/citadel

Field 'K' is now 'rep2=' in the citadel headers

Merge branch 'master' of ssh://git.citadel.org/appl/gitroot/citadel

SMTP-Client: Remove concurrency check; its done in do_housekeeping()

housekeeping: time() is expensive. don't use it inside a mutex.

Updated the comments in serv_smtp.c to reflect that it is strictly a server now.

Added an IDEN code for our z-push backend

Resync with launchpad translations

update french translation for setup as proposed by debian bug#672747

tnx Nicolas Delvaux & David Prévot

configure: check for splice() system call; sendfile isn't all apropriate in older linux or debian k/freebsd

MessageView: don't expose the user to json syntax.

RSSClient: stop curl watchers on emergency shutdown.

DeleteSmtpOutMsg(): we don't have a Queu-Item anymore; we can't log its id.

one more null-check

fix networker hickup

  - refrain from freeing pointers in functions that don't get double pointers and NULL them afterwards
  - call network_consolidate_spoolout () before writing & free'ing the network.

remove unneeded assignment

fix the rest of the unregister functions

remove double initialization

add some more NULL-pointer-checks; free allocated mem in case of OOM.

when failing by OOM, don't loose any more memory.

Hash: forgot to include some lines into the #if DEBUG

locate_host: initialize hostname buffer so we don't send bullshit to the session identifier.

allocate the buffer we use later on in all cases.

remove unneded assignmnet.

add some nullpointer checks.

remove unneeded assignment.

dbg_PrintHash(): #ifdef DEBUG everything related to pure verbose debugging.

remove duplicate assignments; don't compare twice if first is successfull.

remove unneeded assignment.

Fix all *Unregister*Hook() functions.

CtdlUnregisterRoomHook(): second try.

CtdlUnregisterRoomHook(): fix possible nullpointer deref

- dump_message() isn't used anymore; remove it.
- fix potential NULL-pointer dereferencing.

CtdlOutputMsg(): if we found that the attachment which was demanded isn't there, we don't need further actions and should return that error.

After checking for NULL-Pointer exit with failure.

SMTP-Client: another possible NULL-Pointer access.

Wrap NULL-Protection around whole section; the pointer is accessed inbetween too.

Fix loading of message template; cut'n'paste code from webcit. Fix leak.

make shure all of the struct is initialized properly if the DB doesn't give us enough to fill it.

remove unneeded initialisation; separate declaration and initialisation.

Add NULL-Pointer check before starting the work.

In this place we don't care about the result.

validate_recipients(): just remember the mailtype we're finaly interested in.

SMTP-Client: catch error of us failing to allocate the new context.

Event client: add missing forward declaration of StopCurlWatchers

SMTP-Server: Assert with backtrace instead of running in a null pointer situation; this is probably a leftover from a core bug.

RSSClient: actualy use the precalculated length for the StrBuf allocation

OpenID Client: remove duplicate assignment.

Networker:

  - network_learn_topology (): fix prepending of new netmap item to linked list
  - fix removal of spool files not associated anymore

Networker: remove unneeded assignment

remove unused assignments

fix possible buffer underrun (by 1;-)

Check for NULL pointer before passing it into qsort & free

CURL: in this errorcase we don't have a context and have to use another syslog alias. tnx CST

LDAP: dnon't read the reply if we're not interested in. tnx bzed.

Crypto: remove unneded initialization pointed out by clang static analyzer.

Calendar: fix some uninitialized value conditions pointed out by clang static analyzer

fix possible nullpointer dereferenciation; tnx to the clang static analyzer

Remove duplicate line..

clang static analyzer misses that cdb_abort() aborts the process. make it clear we don't march on here.

as the clang static analyser points out, the | is useless here.

Remove unneeded code found by the clang static analyzer

Event-Client: fix to connect logic

  - when connecting delayed also set the connect start timeout and stop the connection faied timer.

DB: query keys are not written by berkeley db. believe me.

Curl-Eventclient: in this errorcase we don't have the context.

its staying libcitdale2

Don't mess around with the so-name for no reason.

Versioning: make the lib 8.x; cleanup.

POP3Client: stop client / db watchers before switching the context.