]> Zhao Yanbai Git Server - minix.git/commitdiff
libc: resolve minix clang warnings 54/2854/1
authorDavid van Moolenbroek <david@minix3.org>
Tue, 30 Sep 2014 20:35:56 +0000 (20:35 +0000)
committerDavid van Moolenbroek <david@minix3.org>
Tue, 30 Sep 2014 20:35:56 +0000 (20:35 +0000)
Change-Id: If6c42f7346cc1b00b387ae8d3b4f0df3ffb0244f

20 files changed:
minix/include/minix/minlib.h
minix/lib/libc/gen/configfile.c
minix/lib/libc/gen/mtab.c
minix/lib/libc/gen/servxcheck.c
minix/lib/libc/gen/sethostname.c
minix/lib/libc/gen/stderr.c
minix/lib/libc/sys/fcntl.c
minix/lib/libc/sys/gcov_flush_sys.c
minix/lib/libc/sys/ioctl.c
minix/lib/libc/sys/ptrace.c
minix/lib/libc/sys/recvfrom.c
minix/lib/libc/sys/sendmsg.c
minix/lib/libc/sys/sendto.c
minix/lib/libc/sys/setpgid.c
minix/lib/libc/sys/setrlimit.c
minix/lib/libc/sys/sizeup.c
minix/lib/libc/sys/socket.c
minix/lib/libc/sys/socketpair.c
minix/lib/libc/sys/sprofile.c
minix/lib/libc/sys/vectorio.c

index 52059eb5a68942bf73593d0eb54e572eb41c810a..e958a5b48dfe2a17b7b7d64a33e27ec4f16b0198 100644 (file)
@@ -15,6 +15,9 @@ void _cpuid(u32_t *eax, u32_t *ebx, u32_t *ecx, u32_t *edx);
 int load_mtab(char *_prog_name);
 int get_mtab_entry(char dev[PATH_MAX], char mount_point[PATH_MAX],
                        char type[MNTNAMELEN], char flags[MNTFLAGLEN]);
+int servxcheck(unsigned long peer, const char *service,
+       void (*logf)(int pass, const char *name));
+char *servxfile(const char *file);
 
 /* read_tsc() and friends */
 void read_tsc(u32_t *hi, u32_t *lo);
index 4c6300fea6eccec64c4caf783e0817b5062c3eae..2907f15ffe18dfd1addd804ce95a96a2c8604083 100644 (file)
@@ -149,7 +149,7 @@ static void skipwhite(void)
     }
 }
 
-static void parse_err(void)
+static void __dead parse_err(void)
 /* Tell user that you can't parse past the current character. */
 {
     char sc[2];
index 88cd1869e2e6ac5786482e762265a76e1cdb1384..a117f81c2d9a5c34954722ca725d94f93839e10f 100644 (file)
@@ -42,7 +42,7 @@
 
 #define BUF_SIZE   512           /* size of the /etc/mtab buffer */
 
-char *etc_mtab = "/etc/mtab";    /* name of the /etc/mtab file */
+const char *etc_mtab = "/etc/mtab";      /* name of the /etc/mtab file */
 static char mtab_in[BUF_SIZE+1];  /* holds /etc/mtab when it is read in */
 static char *iptr = mtab_in;     /* pointer to next line to feed out. */
 
index 47d2caca9606800a6143da2b2a62fd0c76bef6a6..55c61331e40019f045bc63baa0aa2921c926bdcf 100644 (file)
@@ -24,6 +24,8 @@
 #include <net/gen/inet.h>
 #include <net/gen/socket.h>
 #include <netdb.h>
+#include <arpa/inet.h>
+#include <minix/minlib.h>
 
 /* Default service access file. */
 static const char *path_servacces = _PATH_SERVACCES;
@@ -76,12 +78,12 @@ static int netspec(char *word, ipaddr_t *addr, ipaddr_t *mask)
     if ((slash= strchr(word, '/')) == NULL) slash= S32;
 
     *slash= 0;
-    r= inet_aton(word, addr);
+    r= inet_aton(word, (struct in_addr *)addr);
     *slash++= '/';
     if (!r) return 0;
 
     r= 0;
-    while ((*slash - '0') < 10u) {
+    while ((unsigned int)(*slash - '0') < 10u) {
        r= 10*r + (*slash++ - '0');
        if (r > 32) return 0;
     }
@@ -148,7 +150,7 @@ static int get_name(ipaddr_t addr, char *name)
            }
        }
     }
-    strcpy(name, inet_ntoa(addr));
+    strcpy(name, inet_ntoa(*(struct in_addr *)&addr));
     return 0;
 }
 
index 5cc3806b993937aef18aafc44b44475dc1eaab8e..7d8a64ba629455e8d8b1897b04eee6101c0dd0fd 100644 (file)
@@ -5,6 +5,7 @@
 #include <sys/types.h>
 #include <fcntl.h>
 #include <stdlib.h>
+#include <stdio.h>
 #include <string.h>
 #include <unistd.h>
 #include <errno.h>
@@ -38,6 +39,7 @@ int sethostname(const char *buf, size_t len)
 
        r = write(fd, buf, len);
        tmperr = errno;
+       fsync(fd);
        close(fd);
 
        if (r == -1) {
@@ -46,7 +48,7 @@ int sethostname(const char *buf, size_t len)
                return -1;
        }
 
-       if (r < len) {
+       if ((size_t)r < len) {
                unlink(name);
                errno = ENOSPC;
                return -1;
index fd7f65b4b7b67b9170e71a547ddeb85bb36ae7ee..63e802feff47bcdf88cb29bd2f7291e0e2d9618f 100644 (file)
@@ -1,6 +1,7 @@
 #include <lib.h>
 #include <sys/types.h>
 #include <unistd.h>
+#include <minix/minlib.h>
 
 void std_err(const char *s)
 {
index 496dd9a5bc7af60f5e0389848ef1ee878945df38..10b99058f7ef623cd9cd678e9fa74d44efa4110c 100644 (file)
@@ -29,7 +29,7 @@ int fcntl(int fd, int cmd, ...)
      case F_SETLK:
      case F_SETLKW:
      case F_FREESP:
-       m.m_lc_vfs_fcntl.arg_ptr = va_arg(argp, struct flock *);
+       m.m_lc_vfs_fcntl.arg_ptr = (vir_bytes)va_arg(argp, struct flock *);
        break;
   }
 
index 2382d6b0aaed9ad5991845c999eebe53c1cbf4f1..de7d63b3b6aae94d05e5d7a33fcdf1324770565b 100644 (file)
@@ -7,7 +7,7 @@ int gcov_flush_svr(char *buff, int buff_sz, int server_nr)
        message m;
 
        memset(&m, 0, sizeof(m));
-       m.m_lc_vfs_gcov.buff_p = buff;
+       m.m_lc_vfs_gcov.buff_p = (vir_bytes)buff;
        m.m_lc_vfs_gcov.buff_sz = buff_sz;
        m.m_lc_vfs_gcov.pid = server_nr;
 
index b2211d004c246f9b60785771f41c6a23e12e57a3..f1eb9cd4e63d636bf5e165b0f62248ed910d2ed1 100644 (file)
@@ -71,7 +71,7 @@ int     ioctl(int fd, unsigned long request, ...)
                break;
        default:
                /* Keep original as-is */
-               addr = data;
+               addr = (vir_bytes)data;
                break;
   }
 
index 9dd63a9d3aa3e301a3f530fbe74db840aa39f77a..01ae974839b8882104c76b1a7e6d05ae8437302e 100644 (file)
@@ -12,7 +12,7 @@ int ptrace(int req, pid_t pid, void *addr, int data)
   memset(&m, 0, sizeof(m));
   m.m_lc_pm_ptrace.pid = pid;
   m.m_lc_pm_ptrace.req = req;
-  m.m_lc_pm_ptrace.addr = addr;
+  m.m_lc_pm_ptrace.addr = (vir_bytes)addr;
   m.m_lc_pm_ptrace.data = data;
   if (_syscall(PM_PROC_NR, PM_PTRACE, &m) < 0) return(-1);
 
index 58bba5e461b7b227a4900f25605de8bfb8063837..05a7118392a2e030b18fa59d240e5d3acfc62f21 100644 (file)
@@ -19,7 +19,6 @@
 #include <net/gen/udp_io.h>
 
 #include <net/gen/ip_hdr.h>
-#include <net/gen/icmp_hdr.h>
 
 #define DEBUG 0
 
@@ -88,15 +87,14 @@ ssize_t recvfrom(int sock, void *__restrict buffer, size_t length,
 
        {
                ip_hdr_t *ip_hdr;
-                       int ihl, rd;
-               icmp_hdr_t *icmp_hdr;
+               int rd;
                struct sockaddr_in sin;
 
                rd = read(sock, buffer, length);
 
                if(rd < 0) return rd;
 
-               assert(rd >= sizeof(*ip_hdr));
+               assert((size_t)rd >= sizeof(*ip_hdr));
 
                ip_hdr= buffer;
 
@@ -108,8 +106,8 @@ ssize_t recvfrom(int sock, void *__restrict buffer, size_t length,
                        sin.sin_addr.s_addr= ip_hdr->ih_src;
                        sin.sin_len= sizeof(sin);
                        len= *address_len;
-                       if (len > sizeof(sin))
-                               len= sizeof(sin);
+                       if ((size_t)len > sizeof(sin))
+                               len= (int)sizeof(sin);
                        memcpy(address, &sin, len);
                        *address_len= sizeof(sin);
                }
@@ -237,7 +235,7 @@ static ssize_t _udp_recvfrom(int sock, void *__restrict buffer, size_t length,
                return -1;
        }
 
-       assert(r >= sizeof(*io_hdrp));
+       assert((size_t)r >= sizeof(*io_hdrp));
        length= r-sizeof(*io_hdrp);
 
        io_hdrp= buf;
index c0a342b6ac314be4809e5bb4b9d01280df5753d7..b13d8a2518db8c04273b112a045d723b073bf75b 100644 (file)
@@ -3,6 +3,7 @@
 
 #include <errno.h>
 #include <stdlib.h>
+#include <string.h>
 #include <sys/ioctl.h>
 #include <sys/ioc_net.h>
 #include <sys/socket.h>
index d7d846913ccdcd1bf0db7a239e4a4b6ccf783f7c..8370773d4a21145720c93af0970e2d952f04e6cf 100644 (file)
@@ -13,7 +13,6 @@
 
 #include <net/gen/in.h>
 #include <net/gen/ip_hdr.h>
-#include <net/gen/icmp_hdr.h>
 #include <net/gen/tcp.h>
 #include <net/gen/tcp_io.h>
 #include <net/gen/udp.h>
@@ -78,8 +77,6 @@ ssize_t sendto(int sock, const void *message, size_t length, int flags,
 
        {
                ip_hdr_t *ip_hdr;
-               int ihl;
-               icmp_hdr_t *icmp_hdr;
                struct sockaddr_in *sinp;
 
                sinp = (struct sockaddr_in *) __UNCONST(dest_addr);
@@ -199,7 +196,7 @@ static ssize_t _udp_sendto(int sock, const void *message, size_t length,
                errno= t_errno;
                return -1;
        }
-       assert(r == buflen);
+       assert((size_t)r == buflen);
        free(buf);
        return length;
 }
index b007ba532dad77a7836b95710aba40480b1eee72..b724fa44c20537533c607ddd41906b47dbb30fef 100644 (file)
@@ -11,7 +11,7 @@
  */
 int setpgid(pid_t pid, pid_t pgid)
 {
-       pid_t _pid, _pgid, sid, cpid;
+       pid_t _pid, _pgid, cpid;
 
        _pid = pid;
        _pgid = pgid;
index ffa28edb676c51f67dd5d2dfffe90d4716ece708..8e94f31b2feb5485118fcfd8a51f79cdd302d6f0 100644 (file)
@@ -9,8 +9,7 @@
 /* Simple stub for now. */
 int setrlimit(int resource, const struct rlimit *rlp)
 {
-       rlim_t limit;
-       
+
        switch (resource)
        {
                case RLIMIT_CPU:
@@ -35,4 +34,3 @@ int setrlimit(int resource, const struct rlimit *rlp)
 
        return 0;
 }
-
index 41c0245b1a4418f90b1d99486cf814b68065bc17..11df1e96b5e17781f5c9a9b325f740cd01453572 100644 (file)
@@ -17,6 +17,7 @@
 #include <minix/type.h>
 #include <minix/minlib.h>
 #include <minix/partition.h>
+#include <sys/ioctl.h>
 #include <sys/ioc_disk.h>
 
 #include <unistd.h>
index f73828a0187ad41838c860975d2d0d2c3b3b4640..6d5350da5e8118aaad257656f7fa1441170808f8 100644 (file)
@@ -9,9 +9,11 @@ __weak_alias(socket, __socket30)
 #include <fcntl.h>
 #include <signal.h>
 #include <stdio.h>
+#include <string.h>
 #include <unistd.h>
 #include <sys/socket.h>
 
+#include <sys/ioctl.h>
 #include <sys/ioc_net.h>
 #include <net/hton.h>
 #include <net/gen/in.h>
@@ -148,8 +150,7 @@ static int _udp_socket(int type, int protocol)
 
 static int _raw_socket(int type, int protocol)
 {
-       int r, fd, t_errno, flags = O_RDWR;
-       struct sockaddr_in sin;
+       int fd, flags = O_RDWR;
        nwio_ipopt_t ipopt;
        int result;
 
index aa07c7399a1b1fbdb9662497294f3c6b5c9a382b..118e6e40b24f2be6bb1fcd00589ff543bf84e94a 100644 (file)
@@ -6,6 +6,7 @@
 #include <fcntl.h>
 #include <unistd.h>
 #include <net/netlib.h>
+#include <sys/ioctl.h>
 #include <sys/ioc_net.h>
 #include <sys/socket.h>
 #include <sys/stat.h>
index 9f554a49b2a42900ccc9cc053e29dbf2818df22c..0fe27c43f28fe8a7e6a71f888883e8eca14fb639 100644 (file)
@@ -23,8 +23,8 @@ int sprofile(int action,
   m.m_lc_pm_sprof.mem_size     = size;
   m.m_lc_pm_sprof.freq         = freq;
   m.m_lc_pm_sprof.intr_type    = type;
-  m.m_lc_pm_sprof.ctl_ptr      = ctl_ptr;
-  m.m_lc_pm_sprof.mem_ptr      = mem_ptr;
+  m.m_lc_pm_sprof.ctl_ptr      = (vir_bytes)ctl_ptr;
+  m.m_lc_pm_sprof.mem_ptr      = (vir_bytes)mem_ptr;
 
   return _syscall(PM_PROC_NR, PM_SPROF, &m);
 }
index fa14a37b826fc125d77f2a4e57da6cf12e9fd778..6639ea98a6e395333b1d271042c5e80cb4c53ec5 100644 (file)
@@ -39,7 +39,9 @@ static ssize_t vectorio_buffer(int fildes, const struct iovec *iov,
                        while (copied < r)
                        {
                                assert(iovidx < iovcnt);
-                               len = MIN(r - copied, iov[iovidx].iov_len);
+                               len = iov[iovidx].iov_len;
+                               if (len > r - copied)
+                                       len = r - copied;
                                memcpy(iov[iovidx++].iov_base, buffer + copied, len);
                                copied += len;
                        }