Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Segmentation Fault in dnsmasq #194

Closed
brianewell opened this issue Mar 28, 2013 · 13 comments
Closed

Segmentation Fault in dnsmasq #194

brianewell opened this issue Mar 28, 2013 · 13 comments

Comments

@brianewell
Copy link

@brianewell brianewell commented Mar 28, 2013

Getting a segmentation fault from dnsmasq as distributed through the package manager on the following:

"Live Image": "20130321T213641Z",
"CPU Type": "AMD Turion(tm) II Neo N54L Dual-Core Processor"

I would post the core dump, but I doubt that's what github had in mind when it comes to an "image".

@rmustacc

This comment has been minimized.

Copy link
Contributor

@rmustacc rmustacc commented Mar 28, 2013

dnsmasq comes from pkgsrc. So it'd really help if you told us what dataset you're on, and either steps to reproduce this or access to the core file will be necessary. Without any additional debugging information there isn't much we can do.

@brianewell

This comment has been minimized.

Copy link
Author

@brianewell brianewell commented Mar 28, 2013

Sure. Dataset is smartos 1.9.1 base64 (cf7e2f40-9276-11e2-af9a-0bad2233fb0b)
dnsmasq version as reported by pkgin is 2.64
Steps to reproduce are:

pkgin in dnsmasq
svcadm enable dnsmasq:default

checking the log reports "Method "start" failed due to signal SEGV.", and manually running /opt/local/sbin/dnsmasq gives the segmentation fault. How would you like me to deliver this core dump?

@brianewell

This comment has been minimized.

Copy link
Author

@brianewell brianewell commented Mar 28, 2013

@AlainODea

This comment has been minimized.

Copy link
Contributor

@AlainODea AlainODea commented Mar 28, 2013

I had the exact same issue. It also occurs in a 32-bit base (60ed3a3e-92c7-11e2-ba4a-9b6d5feaa0c4) image.

less $(svcs -L dnsmasq) shows:

[ Mar 28 12:07:18 Disabled. ]
[ Mar 28 12:07:18 Rereading configuration. ]
[ Mar 28 12:08:05 Enabled. ]
[ Mar 28 12:08:05 Executing start method ("/opt/local/sbin/dnsmasq -C /opt/local/etc/dnsmasq.conf"). ]

dnsmasq: cannot open or create lease file /var/cache/dnsmasq.leases: No such file or directory
[ Mar 28 12:08:05 Method "start" exited with status 3. ]
[ Mar 28 12:08:05 Executing start method ("/opt/local/sbin/dnsmasq -C /opt/local/etc/dnsmasq.conf"). ]

dnsmasq: cannot open or create lease file /var/cache/dnsmasq.leases: No such file or directory
[ Mar 28 12:08:05 Method "start" exited with status 3. ]
[ Mar 28 12:08:05 Executing start method ("/opt/local/sbin/dnsmasq -C /opt/local/etc/dnsmasq.conf"). ]

dnsmasq: cannot open or create lease file /var/cache/dnsmasq.leases: No such file or directory
[ Mar 28 12:08:05 Method "start" exited with status 3. ]
[ Mar 28 12:14:32 Leaving maintenance because clear requested. ]
[ Mar 28 12:14:32 Enabled. ]
[ Mar 28 12:14:32 Executing start method ("/opt/local/sbin/dnsmasq -C /opt/local/etc/dnsmasq.conf"). ]
[ Mar 28 12:14:32 Method "start" failed due to signal SEGV. ]
[ Mar 28 12:14:47 Leaving maintenance because clear requested. ]
[ Mar 28 12:14:47 Enabled. ]
[ Mar 28 12:14:47 Executing start method ("/opt/local/sbin/dnsmasq -C /opt/local/etc/dnsmasq.conf"). ]
[ Mar 28 12:14:47 Method "start" failed due to signal SEGV. ]

Here is my /opt/local/etc/dnsmasq.conf:

dhcp-match=ipxe,175
dhcp-range=10.102.0.100,10.102.0.200
dhcp-boot=net:#ipxe,undionly.kkpxe.0
dhcp-boot=smartos.ipxe

Just running /opt/local/sbin/dnsmasq dies with Segmentation Fault (core dumped).

I will try with a default dnsmasq configuration and report back.

@brianewell

This comment has been minimized.

Copy link
Author

@brianewell brianewell commented Mar 28, 2013

I found the behavior was the same with a modified or stock config

@AlainODea

This comment has been minimized.

Copy link
Contributor

@AlainODea AlainODea commented Mar 28, 2013

@brian-ewell thanks. That's good to know :)

@brianewell

This comment has been minimized.

Copy link
Author

@brianewell brianewell commented Apr 4, 2013

Attempted to compile this package from source (dnsmasq-2.65) and got another core dump

@jperkin

This comment has been minimized.

Copy link
Contributor

@jperkin jperkin commented Apr 4, 2013

I have found the bug, and will update the packages.

@AlainODea

This comment has been minimized.

Copy link
Contributor

@AlainODea AlainODea commented Apr 4, 2013

Thank you Jonathan :)

On 2013-04-04, at 9:54, Jonathan Perkin notifications@github.com wrote:

I have found the bug, and will update the packages.


Reply to this email directly or view it on GitHub.

@brianewell

This comment has been minimized.

Copy link
Author

@brianewell brianewell commented Apr 4, 2013

Thank you! I know this is probably a fringe package for this distribution, thanks for being on top of fixing it. Let me know when it's up and I'll close the issue after testing.

@brianewell

This comment has been minimized.

Copy link
Author

@brianewell brianewell commented Apr 4, 2013

Have been in conversation with the author or dnsmasq, he brought up this: http://www.thekelleys.org.uk/gitweb/?p=dnsmasq.git;a=commit;h=71c73ac17c7d0908621ef031d82f8e8bc44933d2

@jperkin

This comment has been minimized.

Copy link
Contributor

@jperkin jperkin commented Apr 4, 2013

Yeh, I came up with an independent fix and then found that when trying to submit it upstream. I've taken that patch as well as a follow-up change as per http://mail-index.netbsd.org/pkgsrc-changes/2013/04/04/msg087160.html

@jperkin

This comment has been minimized.

Copy link
Contributor

@jperkin jperkin commented Apr 5, 2013

This is now starting to appear in repositories, and should be in all of them within the next few hours. The version you are looking for is dnsmasq-2.64nb1.

@jperkin jperkin closed this Apr 5, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants
You can’t perform that action at this time.