Fix return code check inversion introduced in last commit

Mark return code of write() as intentionally unused

fix implicit fallthrough spelling

This commit fixes the syntax of the implicit fallthrough comments, in
order to be matched by GCC (and probably other compilers as well) with
the following regular expression:

[ \t.!]*([Ee]lse,? |[Ii]ntentional(ly)? )? fall(s | |-)?thr(ough|u)[
\t.!]*(-[^\n\r]*)?

See: https://gcc.gnu.org/onlinedocs/gcc/Warning-Options.html#index-Wimplicit-fallthrough=

opentracker.c: check set*id return values

This commit fix the following similar warnings:

    opentracker.c:562:7: warning: ignoring return value of ‘setegid’
    declared with attribute ‘warn_unused_result’ [-Wunused-result]
    […]

The man page of these functions ask users to explicitly check the return
value in case of any error happening.

Fix typo. Thanks to Romain Porte

Obey DESTDIR macro. Thanks to Romain Porte

Using strip from enviroment to allow different strippers

Make accesslists work again by testing the actual result of bsearch. Thanks to Hanno

Fix two issues when splitting an iovec to large iobatches

Handle eagain case in trywrite

Add newline to debug string

Merge branch 'split-iobatches' of erdgeist.org:opentracker into split-iobatches

Remove old mman header

Turn mmaps into malloc

Retry successful writes immediately

Split huge iovecs over multiple io_batches

gzip iovecs always end on the boundary and don't need to be fixed

Add a handler for the EAGAIN reason code to iob_send

Rework fullscrape worker, unifying non-gzip and gzip code was a bad idea

accesslist checker should not operate on an empty list

srandomdev is not available on linuxes by default

Use arc4random whereever we need strong entropy

Use dev random by default

Turn random() to nrand48 in inner loop where it is not cryptographically relevant to avoid futex storm. Thanks and credits to Bart Smienk

White space

unmap file if a memory allocation error occurs

Make accesslist reload thread safe. The last commit actually would make a free possible while another thread was bsearching that memory

Try accessing the access lists without locks by making the replacement process as atomic as possible.

De-bottleneck mutex access code

Remove debug code for self pipes

Tell libowfat about EWOULDBLOCK conditions

register both ends of self pipe to libowfat in order to use them

break belongs inside the guard

incorporate a more verbose ascii dump, provided by Tom <tom@foscore.com>

Get rid of some warnings regarding header guards

Avoid double free when iob_addbuf_free fails, thanks to Sami Farin

Forgot to include one error code. Thanks to Lars Seipel

dont count scrapes as full scrapes, fall through fail

Accept was counted twice

Add missing enum value. Thanks to zhangdongmao

Increase number of locks to accomodate for more cores on contemporary CPUs

Avoid reacting to our own error packets, should the happen to end up on our interface with a spoofed source IP. Thanks to zorun.

Fix typo in stat reporting, use udp and not tcp counter. Thanks to zebulon

Protect static data by mutex

Only kick off udp handlers, when everything is initialised

Initialise all values of aes key

Fix immediate overwrite in the X-forwarded-for passing case. Thanks to Alexander Luetjen.

Fix spelling error ;)

Dont make srandomdev the default, it exists only in BSDs

Make woodpecker spotting more usable by adjusting count every 2 minutes

Calculate the old hash for the ip address only, when the current one mismatches

Dont ruin the srandomdev initialisation in trackerlogic's init

Forgot variable declaration

reduce zlib warnings to when there really is something failing

new flag to enforce gzip even if client did not request it

Add rijndael to version report

Remove debug info

Report connection id missmatch count

udp now generates a cryptographically secure token for connecting clients. This is later verified.

Init prng before udp code needs it.

Add connection id missmatch code

Adding rijndael code for udp connection id calculation

Do not always grow the vector, shrink the first iovec to 0 before filling the second one

Need more space for top100

Add a top100 for most popular torrents

Add functionality to distribute udp to several workers

undo last change, it did not work well

fix tiny issue

fix prototype

Try to act upon all udp packets at once

silenced some clang warnings

Open syslog connection before dropping privileges

Prevent infinite loop if whitelist ends in several returns

Chomp all trailing space characters, including new lines, when parsing the config file

Load state only after initialising all structs. Thanks to Michael S. Combs for pointing out the buf

Stop sending tracker id with every packet

Don't only walk through outgoing connections to search for connected peers

Address reconnecting issues

Fixed sync loss bug

Looking into pointer arithmetics issue

tidy up peer count parser code

Fix encoding of peer count in proxy sync packet

enable proxy.debug target

Tidy up sync proxy code

Reset http request input array on error case

the keep-alive loop must not run, if no keep alive is configured or requested for connection

We have been too overly cautious not to read beyond the boundary.

Make whitelist parser more robust against comments. I assumed perfectly arranged white lists until now

** struct ot_workstruct gets ritcher (and will become even ritcher soon).
This is where we encapsulate all per-request data from peer to hash to peer_id, so that it is
available everywhere without passing hundreds of pointers down the stack. Most functions that
do work down the stack now accept an ot_workstruct and some flags. So it can end up in the
stats/event-handler where it will be the default parameter in the future.

** peer_id is now being copied by default and moved to ot_workstruct
So it is available in stats and subsequent functions.

** sync scrape madness is gone
SYNC_SCRAPE was intended to sync tracker state that would normally be lost on restarts i.e.
downloaded counts per torrent. The way was to push it in the tracker cloud after finding all
neighbouring trackers.
This is madness. It never was tested and can be done per tracker by fetching
stats/mode=statedump from time to time and starting opentracker with the -l option later.

** livesync thread has its own ot_workstruct now
So it can behave like ot_udp and ot_http against trackerlogic.c and get rid of the first half
of the embarrassing global variables. The sending half will be fixed soon [tm].

** stats can log completed events
The author recognizes the needs of original content distributors to keep track of the amount
of times a work has been downloaded. While not feasible and used on openbittorrent and other
open and anonymous tracker installations, a tracker user can now choose to send those events
to syslog.

Get rid of SYNC_SCRAPE madness. It's overkill and no one uses it.

Fix white spaces

Add our own time stamps to the completed logs.

Make opentracker's user runtime configurable. Also add more debug output to aid finding problems.

Fix segfault in stats?mode=everything, an additional errorcode was not commited to ot_stats

Only chroot if -d option or tracker.rootdir are set

Fixed including another error message

Make header parsing more efficient, prepare multithreading and keep-alive.

Preparing more efficient header parsing

prevent infinite loop when looking for X-Forwarded-For: Headers

Allow logging of completed events to syslog