]> Zhao Yanbai Git Server - minix.git/commitdiff
Fix various file system warnings 38/938/2
authorDavid van Moolenbroek <david@minix3.org>
Sat, 31 Aug 2013 20:59:44 +0000 (22:59 +0200)
committerLionel Sambuc <lionel@minix3.org>
Tue, 18 Feb 2014 10:25:02 +0000 (11:25 +0100)
Change-Id: Ied10498c3ae14f9f2fd06914f23239df330fa296

lib/libpuffs/link.c
lib/libpuffs/null.c
lib/libpuffs/open.c
lib/libpuffs/puffs.c
lib/libpuffs/read.c
servers/mfs/stadir.c

index 039e820b1760a668d344f6e72c3ebb1e11d8731c..b3a41ae542cdf8d899b48147f9155e8439aec496 100644 (file)
@@ -28,7 +28,7 @@ int fs_ftrunc(void)
   if (end == 0) {
        struct vattr va;
 
-       if (pn->pn_va.va_size == start)
+       if (pn->pn_va.va_size == (u_quad_t) start)
                return(OK);
 
        if (global_pu->pu_ops.puffs_node_setattr == NULL)
@@ -160,8 +160,8 @@ int fs_rdlink()
   char path[PATH_MAX];
   PUFFS_MAKECRED(pcr, &global_kcred);
 
-  copylen = fs_m_in.REQ_MEM_SIZE < UMAX_FILE_POS ?
-               fs_m_in.REQ_MEM_SIZE : UMAX_FILE_POS;
+  copylen = (size_t) fs_m_in.REQ_MEM_SIZE < UMAX_FILE_POS ?
+       (size_t) fs_m_in.REQ_MEM_SIZE : UMAX_FILE_POS;
   
   assert(copylen <= PATH_MAX);
 
index 5e192406290ca7e482940fd16280cc10267025e8..65f176600722649cfd8cc4124f4bf558349e37f8 100644 (file)
@@ -74,8 +74,8 @@ processvattr(const char *path, const struct vattr *va, int regular)
                        return errno;
 
        /* sloppy */
-       if (va->va_atime.tv_sec != (unsigned)PUFFS_VNOVAL
-           || va->va_mtime.tv_sec != (unsigned)PUFFS_VNOVAL) {
+       if (va->va_atime.tv_sec != PUFFS_VNOVAL
+           || va->va_mtime.tv_sec != PUFFS_VNOVAL) {
                /* FIXME: nsec too */
                tbuf.actime = va->va_atime.tv_sec;
                tbuf.modtime = va->va_mtime.tv_sec;
@@ -493,7 +493,7 @@ puffs_null_node_readdir(struct puffs_usermount *pu, puffs_cookie_t opc,
                 /* FIXME: DIRENT_SIZE macro? For now do calculations here */
                int namelen;
                char* cp;
-               int dirent_size;
+               size_t dirent_size;
 
                 entry = readdir(dp);
 
index fd56bcdced9fe4a5eb8ed61448705fdee657265c..f1f7c25086394d5f7b1ceea8ac6afc33a6957f2d 100644 (file)
@@ -318,7 +318,7 @@ int fs_slink()
   if (r != OK) return(r);
   target[fs_m_in.REQ_MEM_SIZE] = '\0';
 
-  if (strlen(target) != fs_m_in.REQ_MEM_SIZE) {
+  if (strlen(target) != (size_t) fs_m_in.REQ_MEM_SIZE) {
        /* This can happen if the user provides a buffer
         * with a \0 in it. This can cause a lot of trouble
         * when the symlink is used later. We could just use
index d6690dec6436648b606fc0432405a8f1387276b7..7bda9e1ca3aca8934cb48ac21752d6b36c1d034b 100644 (file)
@@ -169,7 +169,7 @@ fillvnopmask(struct puffs_ops *pops, uint8_t *opmask)
 
 
 /*ARGSUSED*/
-static void
+__dead static void
 puffs_defaulterror(struct puffs_usermount *pu, uint8_t type,
        int error, const char *str, puffs_cookie_t cookie)
 {
@@ -190,7 +190,8 @@ puffs_getstate(struct puffs_usermount *pu)
 void
 puffs_setstacksize(struct puffs_usermount *pu, size_t ss)
 {
-       long psize, minsize;
+       size_t minsize;
+       int psize;
        int stackshift;
        int bonus;
 
index 707accdbe2644cd3d97f59b83a6055f565f3918e..f3de5ed515f0f183ac48e9e5f8c46aedf0f9c5f9 100644 (file)
@@ -76,7 +76,7 @@ int fs_readwrite(void)
                return(EINVAL);
 
        puffs_vattr_null(&va);
-       if ( (pos + bytes_left) > pn->pn_va.va_size)
+       if ((u_quad_t)(pos + bytes_left) > pn->pn_va.va_size)
                va.va_size = bytes_left + pos;
        va.va_ctime = va.va_mtime = clock_timespec();
        va.va_atime = pn->pn_va.va_atime;
index 85d55d2b1fa1d09d318604811a8f6e7ba43c2b50..6b21d295290903d6443b79e7496fc7c48ab242e6 100644 (file)
@@ -93,7 +93,7 @@ int fs_statvfs()
   struct statvfs st;
   struct super_block *sp;
   int r, scale;
-  uint64_t used;
+  u64_t used;
 
   sp = get_super(fs_dev);