mrash / fwknop

Single Packet Authorization > Port Knocking

Home Page:http://www.cipherdyne.org/fwknop/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

DEF_RUN_DIR uses localstatedir instead of runstatedir

fmarier opened this issue · comments

DEF_RUN_DIR is defined based on SYSRUNDIR which is itself set to localstatedir.

In the Debian package, I've set this to /run but it does look strange to set --localstatedir when there is also a --runstatedir configure variable:

--localstatedir=DIR     modifiable single-machine data [PREFIX/var]
--runstatedir=DIR       modifiable per-process data [LOCALSTATEDIR/run]

It doesn't look like runstatedir is used anywhere and so it doesn't matter that it ends up being set to a non-sensical value (/run/run), but perhaps DEF_RUN_DIR should use that instead of localstatedir?

Hi - can you send a pull request for this one? Not sure if I have it correct. I'm assuming server/fwknopd_common.h needs to be modified at line 72.

I've just created #300 and tested it in the Debian package.