Segmentation fault with pound 4.11 on OpenSUSE about pound

[graygnuorg]

You might be able to reproduce by simply deleting the directory at /usr/local/etc

Ah, you mean system configuration directory! Good point! Thanks a lot. I can reproduce the bug now. A naive fix would be to just throw an error if the directory doesn't exist. However, this might be overly strict, since that directory is not always needed or used. I'll work out a better fix and let you know. Until then, you can use the `-Winclude-dir=DIR` option to work over the bug. E.g. set `-Winclude-dir=/etc` to use `/etc` as a base directory for all relative file names in your configuration, or `-Wno-include-dir`, to use current working directory instead. See the manpage for details. And, while at it, a minor question: you seem to use `/etc/pound.cfg` as your configuration file. Why not compile pound to use that directory by default, then? E.g.: `./configure --sysconfdir=/etc`? Regards, Sergey

[graygnuorg]

==11314== Invalid read of size 4 ==11314== at 0x40E7B0: parse_config_file (config.c:5134) ==11314== by 0x40ECD5: config_parse (config.c:5354) ==11314== by 0x405813: main (pound.c:1000)

I need to have the content of /etc/pound.cfg to debug this. You can send it to my private email, if you prefer.

My crystal ball told me that pound expects a configuration file, even if

Your crystal ball is wrong. There is no such assumption. Regards. Sergey

[geminixdev]

I need to have the content of /etc/pound.cfg to debug this.

It happens with any content.
If there are errors it will tell the errors and segfault after that.

It also happens with a totally empty pound.cfg, or with just some basic lines, such as:

# Start pound as User with Group
User    "pound"
Group   "pound"
Threads     512
LogLevel    3
Alive       5

At first I thought the same, that it is the content of the config file, and I took out content out of it, step by step, until it was empty. Everytime there was the same segmentation fault.

I think it has to do with that the directory /usr/local/etc does not exist yet in a standard OpenSUSE, until some custom software creates it.

You might be able to reproduce by simply deleting the directory at /usr/local/etc and then restarting pound (or even just checking a configuration file, which can be /etc/pound.cfg or anywhere).

This is the line 5134 in config.c which valgrind claims would be causing the segfault:

	      if (include_wd->refcount > 0 && include_wd->fd == AT_FDCWD)

which in a very quick look seems to access a working directory.

So I checked what happens if I delete the empty /usr/local/etc/pound.cfg, and still no segfault.
But if I remove the directory /usr/local/etc then there it is again, the segfault.
And if I create that directory again, just it, no content, that's enough, no more segfault.

So that directory should get created by install, if needed, or better checked in a way that there is no segfault when it does not exist.

Also, thanks to keep pound alive, I use it a lot, and I love the new features you added!

[geminixdev]

Until then, you can use the -Winclude-dir=DIR option to work over the bug. E.g. set -Winclude-dir=/etc to use /etc as a base directory for all relative file names in your configuration, or -Wno-include-dir, to use current working directory instead. See the manpage for details.
And, while at it, a minor question: you seem to use /etc/pound.cfg as your configuration file. Why not compile pound to use that directory by default, then? E.g.: ./configure --sysconfdir=/etc?

Thanks, yes, /etc/pound.cfg has always been the default configuration location on OpenSUSE and Centos, when installing the pound the distribution offered. That's the main reason I use /etc/pound.cfg.

./configure --sysconfdir=/etc seems to be what I need, I will use that in future.

Thanks!