<div dir="ltr"><div>I reviewed this commit:</div><div><br></div><div>commit 1c9f136b57456278ad7aae62b8bae01f01383e1c<br></div><div>Author: Simon Kelley <<a href="mailto:simon@thekelleys.org.uk">simon@thekelleys.org.uk</a>><br>Date:   Tue Jun 15 22:07:59 2021 +0100<br>Man page update, lease times can be given in days or weeks.<br></div><div><br></div><div>And  found the structure 'server_addr4*' was casted from  to 'server*'</div><div><br></div><div>// domain-match.c<br>/* servers need the location in the array to find all the whole<br>set of equivalent servers from a pointer to a single one. */<br>for (count = 0; count < daemon->serverarraysz; count++)</div><div>  if (!(daemon->serverarray[count]->flags & SERV_LITERAL_ADDRESS))</div><div>    daemon->serverarray[count]->arrayposn = count;</div><br><div>But the type server_addr4 has no field 'arraypos'.</div><div><br></div><div>// dnsmasq.h</div><div>/* First three fields must match struct server in next three definitions.. */<br>struct serv_addr4 {<br>  int flags;<br>  char *domain;<br>  struct server *next;<br>  struct in_addr addr;<br>};<br></div><div><br></div><div>Does server_addr4 miss the field 'arraypos'?</div><div><br></div><div>Regards,</div><div>Xingcong Li</div></div>