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.

RSSClient: fix possible buffer underrun

SMTP: free the queue item in the very last function, so we have a valid pointer to the QID for logging.

EV: check the connectionstate on deferred connecting; else we can't detect connections failed by non-existing hosts.

openbsd compatibility: add NS_GET16 workaround.

cleaned up some spelling and grammar in serv_vcard.c

Reverted most of e40ba85133cf6401bda3d6b81a1d144d73d25074 because it broke the protocol semantics and only provided a function that the protocol was already capable of doing. 'verbose_reply' is now gone.  If the client wants a verbose reply, use the 'do_confirm' semantics, which invokes protocol mode 8 (send listing then receive listing) instead of mode 4 (send listing).

Bumped version numbers to 8.11 for upcoming release

Bumped version numbers to 8.11 for upcoming release

When a username/password auth fails, restore the username/password dialog instead of leaving the throbber on the screen.

i guess you can remove the loaded var alltogether then.

Call LoadExpirePolicy() unconditionally, this fixes the apparent snap back to old config when room/floor policy is changed

Still trying to figure out why expire policy settings aren't sticking

wtf?

Added 'go' as hidden input fields to all of the forms in the room editor

Removed er_flags and er_flags2 from editroom().  These were being set and then never sent to the server.

removed a bunch of blank comment lines

Add a 'go' value to the 'edit room configuration' tab so it automatically selects the correct room upon submit

locate_host: add missing defines for openbsd

defines found over at dnssec: https://www.dnssec-tools.org/svn/dnssec-tools/tags/dnsval-1.8/include/validator/resolver.h

Remove duplicate include with wrong #ifs

POP3Client: use URL-Encoder that also encades the '@'

StringBuf: add URL-encoder that also encodes the @ character.

POP3Client: escape URL parts

SMTP-Client: shut down in one chunk

  - splicing up the shutdown sequence into several db-request chunks doesn't work out. do it in one apraoch again.

SMTP-Client: on success strip reply too.

EVENT-Client: set context on attach.

Event-client: add waiting for the server greeting to our error status (preventive..)