Blame SOURCES/unbound-1.6.3-coverity.patch

4359b6
diff --git a/compat/arc4_lock.c b/compat/arc4_lock.c
4359b6
index 0c45ad0..7fb9cdc 100644
4359b6
--- a/compat/arc4_lock.c
4359b6
+++ b/compat/arc4_lock.c
4359b6
@@ -32,7 +32,7 @@
4359b6
  * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
4359b6
  */
4359b6
 #include "config.h"
4359b6
-#define LOCKRET(func) func
4359b6
+#define LOCKRET(func) (void)func
4359b6
 #include "util/locks.h"
4359b6
 
4359b6
 void _ARC4_LOCK(void);
4359b6
diff --git a/daemon/remote.c b/daemon/remote.c
4359b6
index 243d94c..a377aca 100644
4359b6
--- a/daemon/remote.c
4359b6
+++ b/daemon/remote.c
4359b6
@@ -371,7 +371,9 @@ add_open(const char* ip, int nr, struct listen_port** list, int noproto_is_err,
4359b6
 					  (unsigned)cfg_uid, (unsigned)cfg_gid,
4359b6
 					  ip, strerror(errno));
4359b6
 			}
4359b6
-			chmod(ip, (mode_t)(S_IRUSR | S_IWUSR | S_IRGRP | S_IWGRP));
4359b6
+			if (chmod(ip, (mode_t)(S_IRUSR | S_IWUSR | S_IRGRP | S_IWGRP)) == -1)
4359b6
+				verbose(VERB_QUERY, "cannot chmod %s: %s",
4359b6
+					ip, strerror(errno));
4359b6
 #else
4359b6
 			(void)cfg;
4359b6
 #endif
4359b6
@@ -387,7 +389,7 @@ add_open(const char* ip, int nr, struct listen_port** list, int noproto_is_err,
4359b6
 			}
4359b6
 #endif /* USE_WINSOCK */
4359b6
 			log_err("control interface %s:%s getaddrinfo: %s %s",
4359b6
-				ip?ip:"default", port, gai_strerror(r),
4359b6
+				ip, port, gai_strerror(r),
4359b6
 #ifdef EAI_SYSTEM
4359b6
 				r==EAI_SYSTEM?(char*)strerror(errno):""
4359b6
 #else
4359b6
diff --git a/iterator/iter_scrub.c b/iterator/iter_scrub.c
4359b6
index 1bee85c..6bdfe41 100644
4359b6
--- a/iterator/iter_scrub.c
4359b6
+++ b/iterator/iter_scrub.c
4359b6
@@ -437,7 +437,6 @@ scrub_normalize(sldns_buffer* pkt, struct msg_parse* msg,
4359b6
 					rrset->rrset_all_next =
4359b6
 						nx->rrset_all_next;
4359b6
 					nx->rrset_all_next = rrset;
4359b6
-					prev = nx;
4359b6
 				}
4359b6
 			}
4359b6
 
4359b6
diff --git a/services/listen_dnsport.c b/services/listen_dnsport.c
4359b6
index 3b53676..36c67c6 100644
4359b6
--- a/services/listen_dnsport.c
4359b6
+++ b/services/listen_dnsport.c
4359b6
@@ -1072,11 +1072,11 @@ ports_create_if(const char* ifname, int do_auto, int do_udp, int do_tcp,
4359b6
 {
4359b6
 	int s, noip6=0;
4359b6
 #ifdef USE_DNSCRYPT
4359b6
-	int is_dnscrypt = ((strchr(ifname, '@') && 
4359b6
+	const int is_dnscrypt = ((strchr(ifname, '@') &&
4359b6
 			atoi(strchr(ifname, '@')+1) == dnscrypt_port) ||
4359b6
 			(!strchr(ifname, '@') && atoi(port) == dnscrypt_port));
4359b6
 #else
4359b6
-	int is_dnscrypt = 0;
4359b6
+	const int is_dnscrypt = 0;
4359b6
 	(void)dnscrypt_port;
4359b6
 #endif
4359b6
 
4359b6
diff --git a/services/localzone.c b/services/localzone.c
4359b6
index 6bde432..00d044a 100644
4359b6
--- a/services/localzone.c
4359b6
+++ b/services/localzone.c
4359b6
@@ -1079,7 +1079,7 @@ local_zones_tags_lookup(struct local_zones* zones,
4359b6
 	key.name = name;
4359b6
 	key.namelen = len;
4359b6
 	key.namelabs = labs;
4359b6
-	rbtree_find_less_equal(&zones->ztree, &key, &res;;
4359b6
+	(void)rbtree_find_less_equal(&zones->ztree, &key, &res;;
4359b6
 	result = (struct local_zone*)res;
4359b6
 	/* exact or smaller element (or no element) */
4359b6
 	if(!result || result->dclass != dclass)
4359b6
diff --git a/smallapp/unbound-host.c b/smallapp/unbound-host.c
4359b6
index d7a36a2..68f789a 100644
4359b6
--- a/smallapp/unbound-host.c
4359b6
+++ b/smallapp/unbound-host.c
4359b6
@@ -330,6 +330,7 @@ pretty_output(char* q, int t, int c, struct ub_result* result, int docname)
4359b6
 					exit(1);
4359b6
 				}
4359b6
 				printf("%s\n", s);
4359b6
+				free(s);
4359b6
 			} else	printf(" has no %s record", tstr);
4359b6
 			printf(" %s\n", secstatus);
4359b6
 		}