Re: [PATCH] util: fix segfaults when config lacks a section

Martin Schanzenbach <[email protected]> Wed, 24 Jul 2024 16:44:22 +0000
Newsgroups gmane.network.gnunet.devel
Message-ID <[email protected]>
Thank you for the patch.

I think you just may be starting/using the services in a peculiar way
that triggers the configuration API to be called with NULL.

I applied the patch for now, but that does not mean this will run
smoothly (as in, cfg should likely never be NULL anyway with a proper
setup)

BR

On Wed, 2024-07-24 at 18:36 +0200, fence wrote:
> Hi!
> 
> I ran into gnunets services just segfaulting sometimes.
> 
> I ran into the behaviour when running the transport service with the
> transport.conf from the (meson) build output. (with an additional
> PEER 
> section for the private key).
> 
> I think I had similar segfaults with other services too, but don't
> feel 
> like trying to reproduce those tbh.
> 
> Was the NULL check missing due to performance or something?
> 
> Kind regards
> fence
signature.asc (application/pgp-signature, 862 B)
-----BEGIN PGP SIGNATURE-----

iQJIBAABCAAyFiEEPREGPBD5jRS9JNFHCwmY74b1m2oFAmahL2YUHHNjaGFuemVu
QGdudW5ldC5vcmcACgkQCwmY74b1m2r2bg//aKlW2pQcsUd0Te/uUzeBBuFTc56x
xfAH1y15GoSgR/8C9oo+736kC39PKMREmRKJfafaw4B96iPyubV69dmvyC9kO1S/
bSWL8mJDE0jZH2p3kgpas4MVizbaDkuSGGVVLP+K33K1aAToeEkWWA+elmocMWos
CBK+I2TSDsZI/D27SBm1NwXvpNn6fIiTJeI2pXBz0pl9Jny7FwOEC0DB9rZw0jCs
TXRxtkFwFFBE7P4MB5XU8gr+nDZfhdlIraLEJHS9RyTiJP2HksyG5vxXDMOKNkpo
rIAj6xfVY/nGlv9+7p8t1pAkBIHqwWv7y0QTLuo25aeWqHt3HIghfRPgvgW+TZ96
om4YTjYwpRLm29eM7VlMDzac8AOWEhxXecSUC3v5cAeyriIf+IwjOtriLbTwEu9T
cG67EPfDNYNs5LkegCe5dl8oCcaaM7B+H5hb3R0BuI5kukcFI9N1H50VZKsAHxLE
11oSfV+v3SmfLpTabeKBApWYZQLIJd5N3CUVzjh7nWLbqK6oz/EkJeTjT4XQsskD
ji2yvAzTdYmATWqGugtlgwDVj2OFXkX8v3uQdACA+tJyR6l2UHILVbLOgF9bJ6Zq
suG8EN5fV0MpXHZMZPGYmfJZnXlvbJCT7cLeQ4Eb3YO9l9CS/kE8O2W6aCXoPNBr
2DBiVXnqPuunLvo=
=fsYf
-----END PGP SIGNATURE-----