Updated the copyright declaration in several modules, removing any language which encourages the reader to contact the FSF.  Our GPL is open source, not free software.

style

POP3Client: Log the Hostname, not sensitive information.

POP3Client: Check pointer before accessing it

Sharing: add NSYN

  - expose the NSYN command to the webcit room sharing interface.

Add null-check before access.

Sharing: swap sequence to reduce warnings.

Room Edit: add missing closing div

Tidy: remove trailing / which makes all the rest barf

Roomedit - Sharing: remove lines unneeded

  - add portable function to find & remove a section
  - call that function instead of doing it in place.
  - there is no input=input, its input=text

Roomedit - Sharing: swap layout

  - if we change the sequence the tables appear, removing columns work.
  - split i18n text so there is no html inside.

QueueView: use right end conditional

Important message: remove debug blurb

SMTPQueue: add ability to rerun the queue

  - to protect the user from himself force him to read the hint text before actualy triggering SMTP runqueue
  - add the status of the runqueue to the important message to give feedback to the user.

SMTPQueue: fix layout, add missing /div

Layout: allways add div for important message

  - if something special arizes, we show the important_message mechanism.
  - this div so far is only added, if there is something to show.
  - in order to display events at runtime (from ajax replies) we now add it always.
  - hide it if it should be empty
  - only add the autohide-trigger if its shown.

Add define for mailq-view

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;-)