Bug #755
closeddnswatch not working
0%
Description
HI Guys
I noticed that in the latest snapshot dnswatch fails to startup as it seg faults. I managed to get it recompiled with debugging and it is because of /var/etc/dnswatch-ipsec.hosts containing just the hostname. Which is fine except one of the functions, in dnswatch.c, makes use of properties_read().
I am no C programmer but looking at the man file this function requires the contents of the file it is reading to contain a name=value pair - which the contents of dnswatch-ipsec.hosts does not contain. Running the program on a FBSD8 box works fine - so i assume one of the header files(?) on the pfSense box is older and exits when it can't find the '=' in the file it is reading.
So I did my best to remove the properties_read() function and replace it with a fopen() of the file,a fgets for the line (or multiple lines that may be in the file) and assign that to an array which is then used in the while loop. There is most likely a better way of doing this so please do let me know of all my mistakes in the attached patch.
But for now it is working and my DynDNS tunnels on the latest snapshots are all working \o/ (well it seems to be working).
Thx
Warren
Files
Updated by Warren Baker over 14 years ago
Is there a link to your patch? I dont see any changes in the repository?
Updated by Ermal Luçi over 14 years ago
Its not a patch against dnswatch but against the php script that writes the configuration file.
Updated by Warren Baker over 14 years ago
Ok i thought about making that change to the way the file is written but figured that wasn't the most ideal way of handling it although it is the easiest.
Updated by Chris Buechler over 14 years ago
- Category set to IPsec
- Status changed from Feedback to New
- Target version set to 2.0
- Affected Version set to 2.0
dnswatch crashes immediately upon running.
kernel: pid 45001 (dnswatch), uid 0: exited on signal 11 (core dumped)
example contents of /var/etc/dnswatch-ipsec.hosts:
domain.example.com = value
Updated by Chris Buechler over 14 years ago
- Subject changed from dnswatch to dnswatch not working
Updated by Ermal Luçi over 14 years ago
- Status changed from New to Feedback
Can you please test the latest snapshot?
Updated by Warren Baker over 14 years ago
Will check and revert but might take awhile since my remote site is unavailable at the moment :$
Updated by Warren Baker almost 14 years ago
Chris Buechler wrote:
Warren: have you tested this?
Yeah - it was working 100%. I could not test for extended periods of time but for the 2 days I had the remote site up it worked 100% and handled the IP and DynDNS update.
I haven't tested on the later snapshots, will do so once I am back home which is towards the end of the week.
Updated by Chris Buechler almost 14 years ago
- Status changed from Feedback to Resolved
Updated by Warren Baker almost 14 years ago
Unfortunately this is still core dumping for me on latest snaps.
Updated by Chris Buechler almost 14 years ago
- Status changed from Resolved to New
Updated by Warren Baker almost 14 years ago
Just for issue tracking - I have, offlist, mailed gdb output from the core to Ermal.
Updated by Ermal Luçi almost 14 years ago
Pushed a patch to check for null hostnames from the bt it seems null hostname was passed to the gethostbyname which made it crash!
Updated by Ermal Luçi almost 14 years ago
- Status changed from New to Resolved