<div dir="ltr">Thanks for the tip!<div>This is the very first time I use a mailing list and the very first time I submit a patch of any kind.</div><div><br></div><div><div>diff -u dnsmasq-2.66/src/forward.c dnsmasq-2.66-fixed/src/forward.c</div>
<div>--- dnsmasq-2.66/src/forward.c  2013-04-17 14:52:49.000000000 +0200</div><div>+++ dnsmasq-2.66-fixed/src/forward.c    2013-04-21 18:08:31.755258777 +0200</div><div>@@ -369,7 +369,7 @@</div><div><br></div><div> #ifdef HAVE_CONNTRACK</div>
<div>                  /* Copy connection mark of incoming query to outgoing connection. */</div><div>-                 if (option_bool(OPT_CONNTRACK))</div><div>+                 if (udpaddr && option_bool(OPT_CONNTRACK))</div>
<div>                    {</div><div>                      unsigned int mark;</div><div>                      if (get_incoming_mark(udpaddr, dst_addr, 0, &mark))</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">
On Sun, Apr 21, 2013 at 9:50 PM, Matthias Andree <span dir="ltr"><<a href="mailto:matthias.andree@gmx.de" target="_blank">matthias.andree@gmx.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Am <a href="tel:21.04.2013%2018" value="+12104201318">21.04.2013 18</a>:34, schrieb Giacomo Tazzari:<br>
<div class="im">> Hello!<br>
><br>
> I encountered a bug that makes dnsmasq segfaults when the --conntrack<br>
> option is enabled.<br>
> I am not entirely sure how to reproduce it: on my router (an Ubuntu<br>
> server) it seems to occur sometimes when a remote DNS responds with<br>
> ServFail to an A query.<br>
><br>
> Anyway, after compiling with debugging symbols, I figured out how to fix it:<br>
><br>
> $ diff dnsmasq-2.66/src/forward.c dnsmasq-2.66-fixed/src/forward.c<br>
> 372c372<br>
> <                 if (option_bool(OPT_CONNTRACK))<br>
> ---<br>
>>                 if (udpaddr && option_bool(OPT_CONNTRACK))<br>
><br>
<br>
</div>I am not speaking on behalf of Simon,<br>
<br>
but please, for any diff/patch submissions (not only for dnsmasq, but<br>
any open source project I've worked with), always use diff -u, or if<br>
your diff does not offer that option, diff -c, because that way it is<br>
easier to review, and also easier to apply should anything have changed.<br>
<br>
Please re-send your patch in that format to ease Simon's work.<br>
<br>
Thank you.<br>
<br>
<br>
_______________________________________________<br>
Dnsmasq-discuss mailing list<br>
<a href="mailto:Dnsmasq-discuss@lists.thekelleys.org.uk">Dnsmasq-discuss@lists.thekelleys.org.uk</a><br>
<a href="http://lists.thekelleys.org.uk/mailman/listinfo/dnsmasq-discuss" target="_blank">http://lists.thekelleys.org.uk/mailman/listinfo/dnsmasq-discuss</a><br>
</blockquote></div><br></div></div>