One bug fix and two suggestions

Makefile

@@ -1,7 +1,13 @@
-FLAGS=-Wall -Wextra
-all:
-	gcc $(FLAGS) -o dns_proxy dns_proxy.c
-.PHONY : clean
+.PHONY : all clean
+
+CFLAGS ?= -Wall -Wextra
+
+daemon_name = dns_proxy
+
+all: $(daemon_name)
+
+$(daemon_name): $(daemon_name).c
+
 clean :
-	-rm dns_proxy
+	-rm -f $(daemon_name)
 

dns_proxy.c

@@ -146,6 +146,9 @@ void parse_resolv_conf() {
 
 // handle children
 void reaper_handle (int sig) {
+  // use sig as a source for an assignment to circunvent an unused parameter warning
+  // add 1 to sig in the assignment to circunvent a self assign warning
+  sig = sig + 1;
   while (waitpid(-1, NULL, WNOHANG) > 0) { };
 }
 
@@ -173,7 +176,7 @@ void tcp_query(void *query, response *buffer, int len) {
   srand(time(NULL));
 
   // select random dns server
-  in_addr_t remote_dns = inet_addr(dns_servers[rand() % (NUM_DNS - 1)]);
+  in_addr_t remote_dns = inet_addr(dns_servers[rand() % NUM_DNS]);
   memcpy(tmp, "\x05\x01\x00\x01", 4);
   memcpy(tmp + 4, &remote_dns, 4);
   memcpy(tmp + 8, "\x00\x35", 2);