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.

die dv.pc

fix #5571

fix nat-auto port conflict

skip instead of fail hard on testbed link failures in cadet tests

fix build ats

fix HAVE_MEMRCHR

README: we are on python3.7 now

Signed-off-by: ng0 <ng0@n0.is>

Merge branch 'master' of gnunet.org:gnunet

Switch to python3.7 (integration-tests incomplete), continue using python2.7 for gnunet-qr with an incredible annoying workaround for autotools inability to deal with 2 major python versions at the same time

Signed-off-by: ng0 <ng0@n0.is>

DCE

fix #3329

mystery solved?

fix uninit e_hash issue

add missing ignores

use disjoint UNIXPATHs and PORT for NAT

trying to improve things for #5560

ignore generated files, one mroe