Bug#473015: dnsmasq: relay agent options are not echoed back

2008-03-31 Thread Jeremy Lainé
This patch is just a start, there are two remaining calls to clear_packet which I did not feel comfortable with touching: - line 439 : the subsequent call to do_options() forcibly passes NULL as the agent_id, so it seems the Relay Agent info is discarded = why? That code is BOOTP (as

Bug#473015: dnsmasq: relay agent options are not echoed back

2008-03-31 Thread Simon Kelley
Jeremy Lainé wrote: I have just picked up and tried your test release, and it works fine for me, both for a DHCPOFFER and a DHCPACK. Mar 31 11:46:44 blini dnsmasq[9536]: sent size: 6 option: 82:agent-id 01:04:00:00:00:03 Great! Any idea how can I trigger a DHCPNAK to test that case

Bug#473015: dnsmasq: relay agent options are not echoed back

2008-03-31 Thread Jeremy Lainé
Any idea how can I trigger a DHCPNAK to test that case too? Get a lease, then temporarily reduce the size of the dhcp-range so it doesn't include the leased address, then force your client to renew. Thanks for the tip, it works! Mar 31 13:06:17 blini dnsmasq[10760]: DHCPNAK(tunl1)

Bug#473015: dnsmasq: relay agent options are not echoed back

2008-03-30 Thread Simon Kelley
Jeremy Laine wrote: Package: dnsmasq Version: 2.41-2 Severity: normal Tags: patch When dnsmasq replies to DHCP requests containing a Relay Agent Information option, it should include the Relay Agent Information in the reply. Unfortunately, this is not the case because dnsmasq does the

Bug#473015: dnsmasq: relay agent options are not echoed back

2008-03-27 Thread Jeremy Laine
Package: dnsmasq Version: 2.41-2 Severity: normal Tags: patch When dnsmasq replies to DHCP requests containing a Relay Agent Information option, it should include the Relay Agent Information in the reply. Unfortunately, this is not the case because dnsmasq does the following: - copy the Relay