improve the conditional declaration and use of variable to silence cppcheck

diff --git a/lib/isc/netscope.c b/lib/isc/netscope.c
index 39d860990377548fd4d387327e2b0d293852264a..85a25b6f136b24f7f06ed299811bdfd29ca6eff4 100644 (file)
--- a/lib/isc/netscope.c
+++ b/lib/isc/netscope.c
@@ -11,11 +11,6 @@
 
 /*! \file */
 
-#if defined(LIBC_SCCS) && !defined(lint)
-static char rcsid[] =
-       "$Id: netscope.c,v 1.13 2007/06/19 23:47:17 tbox Exp $";
-#endif /* LIBC_SCCS and not lint */
-
 #include <config.h>
 
 #include <isc/string.h>
@@ -28,8 +23,8 @@ isc_netscope_pton(int af, char *scopename, void *addr, isc_uint32_t *zoneid) {
        char *ep;
 #ifdef ISC_PLATFORM_HAVEIFNAMETOINDEX
        unsigned int ifid;
-#endif
        struct in6_addr *in6;
+#endif
        isc_uint32_t zone;
        isc_uint64_t llz;
 
@@ -37,8 +32,6 @@ isc_netscope_pton(int af, char *scopename, void *addr, isc_uint32_t *zoneid) {
        if (af != AF_INET6)
                return (ISC_R_FAILURE);
 
-       in6 = (struct in6_addr *)addr;
-
        /*
         * Basically, "names" are more stable than numeric IDs in terms of
         * renumbering, and are more preferred.  However, since there is no
@@ -48,6 +41,7 @@ isc_netscope_pton(int af, char *scopename, void *addr, isc_uint32_t *zoneid) {
         * interfaces and links.
         */
 #ifdef ISC_PLATFORM_HAVEIFNAMETOINDEX
+       in6 = (struct in6_addr *)addr;
        if (IN6_IS_ADDR_LINKLOCAL(in6) &&
            (ifid = if_nametoindex((const char *)scopename)) != 0)
                zone = (isc_uint32_t)ifid;

diff --git a/lib/isc/unix/entropy.c b/lib/isc/unix/entropy.c
index 847dacdf7c27d392f68229da145d51694652196a..bb493704703a613a8ce05a0b561d0358851d4efe 100644 (file)
--- a/lib/isc/unix/entropy.c
+++ b/lib/isc/unix/entropy.c
@@ -9,8 +9,6 @@
  * information regarding copyright ownership.
  */
 
-/* $Id: entropy.c,v 1.82 2008/12/01 23:47:45 tbox Exp $ */
-
 /* \file unix/entropy.c
  * \brief
  * This is the system dependent part of the ISC entropy API.
@@ -452,11 +450,14 @@ destroyusocketsource(isc_entropyusocketsource_t *source) {
 static isc_result_t
 make_nonblock(int fd) {
        int ret;
-       int flags;
        char strbuf[ISC_STRERRORSIZE];
 #ifdef USE_FIONBIO_IOCTL
        int on = 1;
+#else
+       int flags;
+#endif
 
+#ifdef USE_FIONBIO_IOCTL
        ret = ioctl(fd, FIONBIO, (char *)&on);
 #else
        flags = fcntl(fd, F_GETFL, 0);

diff --git a/lib/isc/unix/socket.c b/lib/isc/unix/socket.c
index 39cdd19d2190f00cbb6fb72a9d7c0bc87dd25749..a04e2d1f2a4595fa525cd88d0c458dcf9bc4ae74 100644 (file)
--- a/lib/isc/unix/socket.c
+++ b/lib/isc/unix/socket.c
@@ -1225,11 +1225,14 @@ select_poke(isc__socketmgr_t *manager, int fd, int msg) {
 static isc_result_t
 make_nonblock(int fd) {
        int ret;
-       int flags;
        char strbuf[ISC_STRERRORSIZE];
 #ifdef USE_FIONBIO_IOCTL
        int on = 1;
+#else
+       int flags;
+#endif
 
+#ifdef USE_FIONBIO_IOCTL
        ret = ioctl(fd, FIONBIO, (char *)&on);
 #else
        flags = fcntl(fd, F_GETFL, 0);
@@ -1311,6 +1314,7 @@ cmsg_space(ISC_SOCKADDR_LEN_T len) {
  */
 static void
 process_cmsg(isc__socket_t *sock, struct msghdr *msg, isc_socketevent_t *dev) {
+#ifdef ISC_NET_BSD44MSGHDR
 #ifdef USE_CMSG
        struct cmsghdr *cmsgp;
 #ifdef ISC_PLATFORM_HAVEIN6PKTINFO
@@ -1319,6 +1323,7 @@ process_cmsg(isc__socket_t *sock, struct msghdr *msg, isc_socketevent_t *dev) {
 #ifdef SO_TIMESTAMP
        void *timevalp;
 #endif
+#endif
 #endif
 
        /*
@@ -1570,6 +1575,7 @@ build_msghdr_send(isc__socket_t *sock, isc_socketevent_t *dev,
                        INSIST((int)sock->dscp == isc_dscp_check_value);
        }
 
+#if defined(IP_TOS) || (defined(IPPROTO_IPV6) && defined(IPV6_TCLASS))
        if ((sock->type == isc_sockettype_udp) &&
            ((dev->attributes & ISC_SOCKEVENTATTR_DSCP) != 0))
        {
@@ -1645,6 +1651,7 @@ build_msghdr_send(isc__socket_t *sock, isc_socketevent_t *dev,
                               sock->sendcmsgbuflen - msg->msg_controllen);
                }
        }
+#endif
 #endif /* USE_CMSG */
 #else /* ISC_NET_BSD44MSGHDR */
        msg->msg_accrights = NULL;