attempt fix #5578

Merge branch 'master' of gnunet.org:gnunet

RPS profiler: Log index of peer

gnunet-bugreport: add checks for pkg_add, some fixes

RPS profiler: Log index of peer

fix README url

README urls

README urls

configure.ac: point to docs.gnunet.org

fix URLs in tutorial

fix URLs in chapter keyconcepts

fix URLs in chapter installation

fix URLs in chapter developer

adjust mantis url

RPS: Adapt to coding style (cosmetic only)

RPS: Use #define instead of hardcoded numbers

RPS: Use stp(n)cpy instead of strncat in loop

RPS service: Use peermap only when non-NULL

RPS service: Adapt to coding guidelines (purely cosmetic)

RPS service: Check existence of peer before destruction

This is only a workaround.
What was causing an issue was that, during the removal of a peer (from
all datastructures), the peer was also removed from the peer_map,
causing the final destruction call to fail.
With this workaround the peer is only destroyed if it is still in the
peer_map.

RPS service: Check if map was already NULLed

RPS service: Get length of buffer, not of string

note

more style fixes

fix uninit issue: bad use of hash map

RPS tests: Provide valgrind logging

RPS test util: Null out pointer to deleted structure

RPS test util: Improve logging

RPS test util: Initialise local memory

fix

support non GNS redirect URIs

Merge branch 'master' of git+ssh://gnunet.org/gnunet

update

fix enable_malicous check

disable TO_FILE logic, causes assertion failures all the time (NULL filename)

fix #5073

port hashmap fixes to other maps, investigated #4905, clearly no longer possible, closing that one

remove index->insert fallback (#5074)

fix #4244

no rps

fully de-monkey

fix build system to actually handle openssl.cnf properly

Merge branch 'master' of gnunet.org:gnunet

fix mistakes which should not have happened

remove obsolete comments

README adjustments

revert sed and fix a typo

clarify README, dependencies

fix typo

gnunet-gns-proxy-setup-ca: conditionally sed openssl and certutil location,
use variable for locations, check for openssl and certutil in configure
phase.

disable rps in unrelated tests

on shutdown, also stop reset tasks

more fs cadet interaction fixes

fix grow behavior of hash map for very large maps

additional santity check for MQ users

remove duplicate valgrind

fix #5586

update ignore file

got integration tests to pass, fixes #5560

note about assertion actually failing

do not strictly require mq non-NULL, as peerinfo service might have died already

fix possibility of NULL result for empty database

do not use rps in consensus test

unset XDG vars in test explicitly in case test is run by hand

extra sanity check for #5582

clean up python logic a bit, remove duplicate print() calls when we have log calls

cleaner py code, fix test_integration_disconnect.py.in by fixing erroneous patch

rexxnor fix for string to digit

add openssl.cnf

fix indentation

Merge branch 'master' of git+ssh://gnunet.org/gnunet

attempt fix #5578

guile 1.6.4 is an optional dependency

Merge branch 'master' of git+ssh://gnunet.org/gnunet

fix path length

filenames can exceed 128 bytes, even in testcases

fix 128

add gnunet homebrew formula

Merge branch 'master' of gnunet.org:gnunet

AUTHORS: Change bugtracker URL and change GPL3 to AGPL3

AUTHORS: Add myself as netbsd maintainer

skip if timeout not installed

Florian Weimer writes:

 Christian Grothoff:

> I'm seeing some _very_ odd behavior with processes hanging on exit (?)
> with GNU libc 2.28-6 on Debian (amd64 threadripper).  This seems to
> happen at random (for random tests, with very low frequency!) in the
> GNUnet (Git master) testsuite when a child process is about to exit.

It looks like you call exit from a signal handler, see
src/util/scheduler.c:

/**
 * Signal handler called for signals that should cause us to shutdown.
 */
static void
sighandler_shutdown ()
{
  static char c;
  int old_errno = errno;        /* backup errno */

  if (getpid () != my_pid)
    exit (1);                   /* we have fork'ed since the signal handler was created,
                                 * ignore the signal, see https://gnunet.org/vfork discussion */
  GNUNET_DISK_file_write (GNUNET_DISK_pipe_handle
                          (shutdown_pipe_handle, GNUNET_DISK_PIPE_END_WRITE),
                          &c, sizeof (c));
  errno = old_errno;
}

In general, this results in undefined behavior because exit (unlike
_exit) is not an async-signal-safe function.

I suspect you either call the exit function while a fork is in progress,
or since you register this signal handler multiple times for different
signals:

  sh->shc_int = GNUNET_SIGNAL_handler_install (SIGINT,
                                               &sighandler_shutdown);
  sh->shc_term = GNUNET_SIGNAL_handler_install (SIGTERM,
                                                &sighandler_shutdown);

one call to exit might interrupt another call to exit if both signals
are delivered to the process.

The deadlock you see was introduced in commit
27761a1042daf01987e7d79636d0c41511c6df3c ("Refactor atfork handlers"),
first released in glibc 2.28.  The fork deadlock will be gone (in the
single-threaded case) if Debian updates to the current
release/2.28/master branch because we backported commit
60f80624257ef84eacfd9b400bda1b5a5e8e7816 ("nptl: Avoid fork handler lock
for async-signal-safe fork [BZ #24161]") there.

But this will not help you.  Even without the deadlock, I expect you
still experience some random corruption during exit, but it's going to
be difficult to spot.

Thanks,
Florian

fix another linker error

Merge branch 'master' of git+ssh://gnunet.org/gnunet

fix assertion failure reported in #5578

more logger

more logging

gnunet_testing.py.in: first set of logging

add debug output to py.in

add logging

cast no longer required for modern curl

make clang shut up about #5573

Merge branch 'master' of gnunet.org:gnunet

text-only variant of svg logo

fix some compiler warnings

fix dist build

fix macos build of dist tarball

Merge branch 'master' of gnunet.org:gnunet

Fix bug reported by schanzen@ where bootstrap is not enough for our python2.7 script.