s3:fix build of old linux quota system and other unixes' quota implementation
[metze/samba/wip.git] / source3 / smbd / quotas.c
index ac6ad9d470614a546e803c4d01ce53a812d16131..43b700935e1f86961935914a24218b5810488699 100644 (file)
@@ -45,7 +45,7 @@
  * Declare here, define at end: reduces likely "include" interaction problems.
  *     David Lee <T.D.Lee@durham.ac.uk>
  */
-bool disk_quotas_vxfs(const pstring name, char *path, SMB_BIG_UINT *bsize, SMB_BIG_UINT *dfree, SMB_BIG_UINT *dsize);
+bool disk_quotas_vxfs(const char *name, char *path, uint64_t *bsize, uint64_t *dfree, uint64_t *dsize);
 
 #endif /* VXFS_QUOTA */
 
@@ -63,13 +63,13 @@ bool disk_quotas_vxfs(const pstring name, char *path, SMB_BIG_UINT *bsize, SMB_B
 #include "samba_linux_quota.h"
 
 typedef struct _LINUX_SMB_DISK_QUOTA {
-       SMB_BIG_UINT bsize;
-       SMB_BIG_UINT hardlimit; /* In bsize units. */
-       SMB_BIG_UINT softlimit; /* In bsize units. */
-       SMB_BIG_UINT curblocks; /* In bsize units. */
-       SMB_BIG_UINT ihardlimit; /* inode hard limit. */
-       SMB_BIG_UINT isoftlimit; /* inode soft limit. */
-       SMB_BIG_UINT curinodes; /* Current used inodes. */
+       uint64_t bsize;
+       uint64_t hardlimit; /* In bsize units. */
+       uint64_t softlimit; /* In bsize units. */
+       uint64_t curblocks; /* In bsize units. */
+       uint64_t ihardlimit; /* inode hard limit. */
+       uint64_t isoftlimit; /* inode soft limit. */
+       uint64_t curinodes; /* Current used inodes. */
 } LINUX_SMB_DISK_QUOTA;
 
 
@@ -95,13 +95,13 @@ static int get_smb_linux_xfs_quota(char *path, uid_t euser_id, gid_t egrp_id, LI
        if (ret)
                return ret;
 
-       dp->bsize = (SMB_BIG_UINT)512;
-       dp->softlimit = (SMB_BIG_UINT)D.d_blk_softlimit;
-       dp->hardlimit = (SMB_BIG_UINT)D.d_blk_hardlimit;
-       dp->ihardlimit = (SMB_BIG_UINT)D.d_ino_hardlimit;
-       dp->isoftlimit = (SMB_BIG_UINT)D.d_ino_softlimit;
-       dp->curinodes = (SMB_BIG_UINT)D.d_icount;
-       dp->curblocks = (SMB_BIG_UINT)D.d_bcount;
+       dp->bsize = (uint64_t)512;
+       dp->softlimit = (uint64_t)D.d_blk_softlimit;
+       dp->hardlimit = (uint64_t)D.d_blk_hardlimit;
+       dp->ihardlimit = (uint64_t)D.d_ino_hardlimit;
+       dp->isoftlimit = (uint64_t)D.d_ino_softlimit;
+       dp->curinodes = (uint64_t)D.d_icount;
+       dp->curblocks = (uint64_t)D.d_bcount;
 
        return ret;
 }
@@ -134,13 +134,13 @@ static int get_smb_linux_v1_quota(char *path, uid_t euser_id, gid_t egrp_id, LIN
        if (ret && errno != EDQUOT)
                return ret;
 
-       dp->bsize = (SMB_BIG_UINT)QUOTABLOCK_SIZE;
-       dp->softlimit = (SMB_BIG_UINT)D.dqb_bsoftlimit;
-       dp->hardlimit = (SMB_BIG_UINT)D.dqb_bhardlimit;
-       dp->ihardlimit = (SMB_BIG_UINT)D.dqb_ihardlimit;
-       dp->isoftlimit = (SMB_BIG_UINT)D.dqb_isoftlimit;
-       dp->curinodes = (SMB_BIG_UINT)D.dqb_curinodes;
-       dp->curblocks = (SMB_BIG_UINT)D.dqb_curblocks;
+       dp->bsize = (uint64_t)QUOTABLOCK_SIZE;
+       dp->softlimit = (uint64_t)D.dqb_bsoftlimit;
+       dp->hardlimit = (uint64_t)D.dqb_bhardlimit;
+       dp->ihardlimit = (uint64_t)D.dqb_ihardlimit;
+       dp->isoftlimit = (uint64_t)D.dqb_isoftlimit;
+       dp->curinodes = (uint64_t)D.dqb_curinodes;
+       dp->curblocks = (uint64_t)D.dqb_curblocks;
 
        return ret;
 }
@@ -160,13 +160,13 @@ static int get_smb_linux_v2_quota(char *path, uid_t euser_id, gid_t egrp_id, LIN
        if (ret && errno != EDQUOT)
                return ret;
 
-       dp->bsize = (SMB_BIG_UINT)QUOTABLOCK_SIZE;
-       dp->softlimit = (SMB_BIG_UINT)D.dqb_bsoftlimit;
-       dp->hardlimit = (SMB_BIG_UINT)D.dqb_bhardlimit;
-       dp->ihardlimit = (SMB_BIG_UINT)D.dqb_ihardlimit;
-       dp->isoftlimit = (SMB_BIG_UINT)D.dqb_isoftlimit;
-       dp->curinodes = (SMB_BIG_UINT)D.dqb_curinodes;
-       dp->curblocks = ((SMB_BIG_UINT)D.dqb_curspace) / dp->bsize;
+       dp->bsize = (uint64_t)QUOTABLOCK_SIZE;
+       dp->softlimit = (uint64_t)D.dqb_bsoftlimit;
+       dp->hardlimit = (uint64_t)D.dqb_bhardlimit;
+       dp->ihardlimit = (uint64_t)D.dqb_ihardlimit;
+       dp->isoftlimit = (uint64_t)D.dqb_isoftlimit;
+       dp->curinodes = (uint64_t)D.dqb_curinodes;
+       dp->curblocks = ((uint64_t)D.dqb_curspace) / dp->bsize;
 
        return ret;
 }
@@ -190,13 +190,13 @@ static int get_smb_linux_gen_quota(char *path, uid_t euser_id, gid_t egrp_id, LI
        if (ret && errno != EDQUOT)
                return ret;
 
-       dp->bsize = (SMB_BIG_UINT)QUOTABLOCK_SIZE;
-       dp->softlimit = (SMB_BIG_UINT)D.dqb_bsoftlimit;
-       dp->hardlimit = (SMB_BIG_UINT)D.dqb_bhardlimit;
-       dp->ihardlimit = (SMB_BIG_UINT)D.dqb_ihardlimit;
-       dp->isoftlimit = (SMB_BIG_UINT)D.dqb_isoftlimit;
-       dp->curinodes = (SMB_BIG_UINT)D.dqb_curinodes;
-       dp->curblocks = ((SMB_BIG_UINT)D.dqb_curspace) / dp->bsize;
+       dp->bsize = (uint64_t)QUOTABLOCK_SIZE;
+       dp->softlimit = (uint64_t)D.dqb_bsoftlimit;
+       dp->hardlimit = (uint64_t)D.dqb_bhardlimit;
+       dp->ihardlimit = (uint64_t)D.dqb_ihardlimit;
+       dp->isoftlimit = (uint64_t)D.dqb_isoftlimit;
+       dp->curinodes = (uint64_t)D.dqb_curinodes;
+       dp->curblocks = ((uint64_t)D.dqb_curspace) / dp->bsize;
 
        return ret;
 }
@@ -205,7 +205,7 @@ static int get_smb_linux_gen_quota(char *path, uid_t euser_id, gid_t egrp_id, LI
  Try to get the disk space from disk quotas (LINUX version).
 ****************************************************************************/
 
-bool disk_quotas(const char *path, SMB_BIG_UINT *bsize, SMB_BIG_UINT *dfree, SMB_BIG_UINT *dsize)
+bool disk_quotas(const char *path, uint64_t *bsize, uint64_t *dfree, uint64_t *dsize)
 {
        int r;
        SMB_STRUCT_STAT S;
@@ -223,29 +223,29 @@ bool disk_quotas(const char *path, SMB_BIG_UINT *bsize, SMB_BIG_UINT *dfree, SMB
        egrp_id = getegid();
 
        /* find the block device file */
-  
+
        if ( sys_stat(path, &S) == -1 )
                return(False) ;
 
-       devno = S.st_dev ;
-  
+       devno = S.st_ex_dev ;
+
        if ((fp = setmntent(MOUNTED,"r")) == NULL)
                return(False) ;
 
        found = False ;
-  
+
        while ((mnt = getmntent(fp))) {
                if ( sys_stat(mnt->mnt_dir,&S) == -1 )
                        continue ;
 
-               if (S.st_dev == devno) {
+               if (S.st_ex_dev == devno) {
                        found = True ;
                        break;
                }
        }
 
        endmntent(fp) ;
-  
+
        if (!found)
                return(False);
 
@@ -306,96 +306,83 @@ bool disk_quotas(const char *path, SMB_BIG_UINT *bsize, SMB_BIG_UINT *dfree, SMB
 try to get the disk space from disk quotas (CRAY VERSION)
 ****************************************************************************/
 
-bool disk_quotas(const char *path, SMB_BIG_UINT *bsize, SMB_BIG_UINT *dfree, SMB_BIG_UINT *dsize)
+bool disk_quotas(const char *path, uint64_t *bsize, uint64_t *dfree, uint64_t *dsize)
 {
-  struct mntent *mnt;
-  FILE *fd;
-  SMB_STRUCT_STAT sbuf;
-  SMB_DEV_T devno ;
-  static SMB_DEV_T devno_cached = 0 ;
-  static pstring name;
-  struct q_request request ;
-  struct qf_header header ;
-  static int quota_default = 0 ;
-  int found ;
-  
-  if ( sys_stat(path,&sbuf) == -1 )
-    return(False) ;
-  
-  devno = sbuf.st_dev ;
-  
-  if ( devno != devno_cached ) {
-    
-    devno_cached = devno ;
-    
-    if ((fd = setmntent(KMTAB)) == NULL)
-      return(False) ;
-    
-    found = False ;
-    
-    while ((mnt = getmntent(fd)) != NULL) {
-      
-      if ( sys_stat(mnt->mnt_dir,&sbuf) == -1 )
-       continue ;
-      
-      if (sbuf.st_dev == devno) {
-       
-       found = True ;
-       break ;
-       
-      }
-      
-    }
-    
-    pstrcpy(name,mnt->mnt_dir) ;
-    endmntent(fd) ;
-    
-    if ( ! found )
-      return(False) ;
-  }
-  
-  request.qf_magic = QF_MAGIC ;
-  request.qf_entry.id = geteuid() ;
-  
-  if (quotactl(name, Q_GETQUOTA, &request) == -1)
-    return(False) ;
-  
-  if ( ! request.user )
-    return(False) ;
-  
-  if ( request.qf_entry.user_q.f_quota == QFV_DEFAULT ) {
-    
-    if ( ! quota_default ) {
-      
-      if ( quotactl(name, Q_GETHEADER, &header) == -1 )
-       return(False) ;
-      else
-       quota_default = header.user_h.def_fq ;
-    }
-    
-    *dfree = quota_default ;
-    
-  }else if ( request.qf_entry.user_q.f_quota == QFV_PREVENT ) {
-    
-    *dfree = 0 ;
-    
-  }else{
-    
-    *dfree = request.qf_entry.user_q.f_quota ;
-    
-  }
-  
-  *dsize = request.qf_entry.user_q.f_use ;
-  
-  if ( *dfree < *dsize )
-    *dfree = 0 ;
-  else
-    *dfree -= *dsize ;
-  
-  *bsize = 4096 ;  /* Cray blocksize */
-  
-  return(True) ;
-  
+       struct mntent *mnt;
+       FILE *fd;
+       SMB_STRUCT_STAT sbuf;
+       SMB_DEV_T devno ;
+       struct q_request request ;
+       struct qf_header header ;
+       int quota_default = 0 ;
+       bool found = false;
+
+       if (sys_stat(path,&sbuf) == -1) {
+               return false;
+       }
+
+       devno = sbuf.st_ex_dev ;
+
+       if ((fd = setmntent(KMTAB)) == NULL) {
+               return false;
+       }
+
+       while ((mnt = getmntent(fd)) != NULL) {
+               if (sys_stat(mnt->mnt_dir,&sbuf) == -1) {
+                       continue;
+               }
+               if (sbuf.st_ex_dev == devno) {
+                       found = frue ;
+                       break;
+               }
+       }
+
+       name = talloc_strdup(talloc_tos(), mnt->mnt_dir);
+       endmntent(fd);
+       if (!found) {
+               return false;
+       }
+
+       if (!name) {
+               return false;
+       }
+
+       request.qf_magic = QF_MAGIC ;
+       request.qf_entry.id = geteuid() ;
+
+       if (quotactl(name, Q_GETQUOTA, &request) == -1) {
+               return false;
+       }
+
+       if (!request.user) {
+               return False;
+       }
+
+       if (request.qf_entry.user_q.f_quota == QFV_DEFAULT) {
+               if (!quota_default) {
+                       if (quotactl(name, Q_GETHEADER, &header) == -1) {
+                               return false;
+                       } else {
+                               quota_default = header.user_h.def_fq;
+                       }
+               }
+               *dfree = quota_default;
+       } else if (request.qf_entry.user_q.f_quota == QFV_PREVENT) {
+               *dfree = 0;
+       } else {
+               *dfree = request.qf_entry.user_q.f_quota;
+       }
+
+       *dsize = request.qf_entry.user_q.f_use;
+
+       if (*dfree < *dsize) {
+               *dfree = 0;
+       } else {
+               *dfree -= *dsize;
+       }
+
+       *bsize = 4096 ;  /* Cray blocksize */
+       return true;
 }
 
 
@@ -426,8 +413,6 @@ bool disk_quotas(const char *path, SMB_BIG_UINT *bsize, SMB_BIG_UINT *dfree, SMB
 #include <rpc/nettype.h>
 #include <rpc/xdr.h>
 
-static int quotastat;
-
 static int my_xdr_getquota_args(XDR *xdrsp, struct getquota_args *args)
 {
        if (!xdr_string(xdrsp, &args->gqa_pathp, RQ_PATHLEN ))
@@ -439,10 +424,14 @@ static int my_xdr_getquota_args(XDR *xdrsp, struct getquota_args *args)
 
 static int my_xdr_getquota_rslt(XDR *xdrsp, struct getquota_rslt *gqr)
 {
+       int quotastat;
+
        if (!xdr_int(xdrsp, &quotastat)) {
                DEBUG(6,("nfs_quotas: Status bad or zero\n"));
                return 0;
        }
+       gqr->status = quotastat;
+
        if (!xdr_int(xdrsp, &gqr->getquota_rslt_u.gqr_rquota.rq_bsize)) {
                DEBUG(6,("nfs_quotas: Block size bad or zero\n"));
                return 0;
@@ -466,8 +455,8 @@ static int my_xdr_getquota_rslt(XDR *xdrsp, struct getquota_rslt *gqr)
        return (1);
 }
 
-/* Restricted to SUNOS5 for the moment, I haven`t access to others to test. */ 
-static bool nfs_quotas(char *nfspath, uid_t euser_id, SMB_BIG_UINT *bsize, SMB_BIG_UINT *dfree, SMB_BIG_UINT *dsize)
+/* Restricted to SUNOS5 for the moment, I haven`t access to others to test. */
+static bool nfs_quotas(char *nfspath, uid_t euser_id, uint64_t *bsize, uint64_t *dfree, uint64_t *dsize)
 {
        uid_t uid = euser_id;
        struct dqblk D;
@@ -481,7 +470,7 @@ static bool nfs_quotas(char *nfspath, uid_t euser_id, SMB_BIG_UINT *bsize, SMB_B
        enum clnt_stat clnt_stat;
        bool ret = True;
 
-       *bsize = *dfree = *dsize = (SMB_BIG_UINT)0;
+       *bsize = *dfree = *dsize = (uint64_t)0;
 
        len=strcspn(mnttype, ":");
        pathname=strstr(mnttype, ":");
@@ -515,15 +504,15 @@ static bool nfs_quotas(char *nfspath, uid_t euser_id, SMB_BIG_UINT *bsize, SMB_B
                goto out;
        }
 
-       /* 
-        * quotastat returns 0 if the rpc call fails, 1 if quotas exist, 2 if there is
+       /*
+        * gqr.status returns 0 if the rpc call fails, 1 if quotas exist, 2 if there is
         * no quota set, and 3 if no permission to get the quota.  If 0 or 3 return
         * something sensible.
-        */   
+        */
 
-       switch ( quotastat ) {
+       switch (gqr.status) {
        case 0:
-               DEBUG(9,("nfs_quotas: Remote Quotas Failed!  Error \"%i\" \n", quotastat ));
+               DEBUG(9,("nfs_quotas: Remote Quotas Failed!  Error \"%i\" \n", gqr.status));
                ret = False;
                goto out;
 
@@ -538,16 +527,16 @@ static bool nfs_quotas(char *nfspath, uid_t euser_id, SMB_BIG_UINT *bsize, SMB_B
        case 3:
                D.dqb_bsoftlimit = 1;
                D.dqb_curblocks = 1;
-               DEBUG(9,("nfs_quotas: Remote Quotas returned \"%i\" \n", quotastat ));
+               DEBUG(9,("nfs_quotas: Remote Quotas returned \"%i\" \n", gqr.status));
                break;
 
        default:
-               DEBUG(9,("nfs_quotas: Remote Quotas Questionable!  Error \"%i\" \n", quotastat ));
+               DEBUG(9,("nfs_quotas: Remote Quotas Questionable!  Error \"%i\" \n", gqr.status ));
                break;
        }
 
        DEBUG(10,("nfs_quotas: Let`s look at D a bit closer... status \"%i\" bsize \"%i\" active? \"%i\" bhard \"%i\" bsoft \"%i\" curb \"%i\" \n",
-                       quotastat,
+                       gqr.status,
                        gqr.getquota_rslt_u.gqr_rquota.rq_bsize,
                        gqr.getquota_rslt_u.gqr_rquota.rq_active,
                        gqr.getquota_rslt_u.gqr_rquota.rq_bhardlimit,
@@ -557,7 +546,7 @@ static bool nfs_quotas(char *nfspath, uid_t euser_id, SMB_BIG_UINT *bsize, SMB_B
        *bsize = gqr.getquota_rslt_u.gqr_rquota.rq_bsize;
        *dsize = D.dqb_bsoftlimit;
 
-       if (D.dqb_curblocks == D.dqb_curblocks == 1)
+       if (D.dqb_curblocks == 1)
                *bsize = 512;
 
        if (D.dqb_curblocks > D.dqb_bsoftlimit) {
@@ -587,7 +576,10 @@ try to get the disk space from disk quotas (SunOS & Solaris2 version)
 Quota code by Peter Urbanec (amiga@cse.unsw.edu.au).
 ****************************************************************************/
 
-bool disk_quotas(const char *path, SMB_BIG_UINT *bsize, SMB_BIG_UINT *dfree, SMB_BIG_UINT *dsize)
+bool disk_quotas(const char *path,
+               uint64_t *bsize,
+               uint64_t *dfree,
+               uint64_t *dsize)
 {
        uid_t euser_id;
        int ret;
@@ -595,84 +587,90 @@ bool disk_quotas(const char *path, SMB_BIG_UINT *bsize, SMB_BIG_UINT *dfree, SMB
 #if defined(SUNOS5)
        struct quotctl command;
        int file;
-       static struct mnttab mnt;
-       static pstring name;
+       struct mnttab mnt;
 #else /* SunOS4 */
        struct mntent *mnt;
-       static pstring name;
 #endif
+       char *name = NULL;
        FILE *fd;
        SMB_STRUCT_STAT sbuf;
-       SMB_DEV_T devno ;
-       static SMB_DEV_T devno_cached = 0 ;
-       static int found ;
+       SMB_DEV_T devno;
+       bool found = false;
 
        euser_id = geteuid();
-  
-       if ( sys_stat(path,&sbuf) == -1 )
-               return(False) ;
-  
-       devno = sbuf.st_dev ;
+
+       if (sys_stat(path,&sbuf) == -1) {
+               return false;
+       }
+
+       devno = sbuf.st_ex_dev ;
        DEBUG(5,("disk_quotas: looking for path \"%s\" devno=%x\n",
                path, (unsigned int)devno));
-       if ( devno != devno_cached ) {
-               devno_cached = devno ;
 #if defined(SUNOS5)
-               if ((fd = sys_fopen(MNTTAB, "r")) == NULL)
-                       return(False) ;
-    
-               found = False ;
-
-               while (getmntent(fd, &mnt) == 0) {
-                       if (sys_stat(mnt.mnt_mountp, &sbuf) == -1)
-                               continue;
-
-                       DEBUG(5,("disk_quotas: testing \"%s\" devno=%x\n",
-                               mnt.mnt_mountp, (unsigned int)devno));
-
-                       /* quotas are only on vxfs, UFS or NFS */
-                       if ( (sbuf.st_dev == devno) && (
-                               strcmp( mnt.mnt_fstype, MNTTYPE_UFS ) == 0 ||
-                               strcmp( mnt.mnt_fstype, "nfs" ) == 0    ||
-                               strcmp( mnt.mnt_fstype, "vxfs" ) == 0 )) { 
-                                       found = True ;
-                                       break;
-                       }
+       if ((fd = sys_fopen(MNTTAB, "r")) == NULL) {
+               return false;
+       }
+
+       while (getmntent(fd, &mnt) == 0) {
+               if (sys_stat(mnt.mnt_mountp, &sbuf) == -1) {
+                       continue;
                }
-    
-               pstrcpy(name,mnt.mnt_mountp) ;
-               pstrcat(name,"/quotas") ;
-               fclose(fd) ;
-#else /* SunOS4 */
-               if ((fd = setmntent(MOUNTED, "r")) == NULL)
-                       return(False) ;
-    
-               found = False ;
-               while ((mnt = getmntent(fd)) != NULL) {
-                       if ( sys_stat(mnt->mnt_dir,&sbuf) == -1 )
-                               continue ;
-                       DEBUG(5,("disk_quotas: testing \"%s\" devno=%x\n", mnt->mnt_dir,(unsigned int)sbuf.st_dev));
-                       if (sbuf.st_dev == devno) {
-                               found = True ;
+
+               DEBUG(5,("disk_quotas: testing \"%s\" devno=%x\n",
+                       mnt.mnt_mountp, (unsigned int)devno));
+
+               /* quotas are only on vxfs, UFS or NFS */
+               if ((sbuf.st_ex_dev == devno) && (
+                       strcmp( mnt.mnt_fstype, MNTTYPE_UFS ) == 0 ||
+                       strcmp( mnt.mnt_fstype, "nfs" ) == 0    ||
+                       strcmp( mnt.mnt_fstype, "vxfs" ) == 0 )) {
+                               found = true;
+                               name = talloc_asprintf(talloc_tos(),
+                                               "%s/quotas",
+                                               mnt.mnt_mountp);
                                break;
-                       }
                }
-    
-               pstrcpy(name,mnt->mnt_fsname) ;
-               endmntent(fd) ;
-#endif
        }
 
-       if ( ! found )
-               return(False) ;
+       fclose(fd);
+#else /* SunOS4 */
+       if ((fd = setmntent(MOUNTED, "r")) == NULL) {
+               return false;
+       }
+
+       while ((mnt = getmntent(fd)) != NULL) {
+               if (sys_stat(mnt->mnt_dir,&sbuf) == -1) {
+                       continue;
+               }
+               DEBUG(5,("disk_quotas: testing \"%s\" devno=%x\n",
+                                       mnt->mnt_dir,
+                                       (unsigned int)sbuf.st_ex_dev));
+               if (sbuf.st_ex_dev == devno) {
+                       found = true;
+                       name = talloc_strdup(talloc_tos(),
+                                       mnt->mnt_fsname);
+                       break;
+               }
+       }
+
+       endmntent(fd);
+#endif
+       if (!found) {
+               return false;
+       }
 
+       if (!name) {
+               return false;
+       }
        become_root();
 
 #if defined(SUNOS5)
-       if ( strcmp( mnt.mnt_fstype, "nfs" ) == 0) {
+       if (strcmp(mnt.mnt_fstype, "nfs") == 0) {
                bool retval;
-               DEBUG(5,("disk_quotas: looking for mountpath (NFS) \"%s\"\n", mnt.mnt_special));
-               retval = nfs_quotas(mnt.mnt_special, euser_id, bsize, dfree, dsize);
+               DEBUG(5,("disk_quotas: looking for mountpath (NFS) \"%s\"\n",
+                                       mnt.mnt_special));
+               retval = nfs_quotas(mnt.mnt_special,
+                               euser_id, bsize, dfree, dsize);
                unbecome_root();
                return retval;
        }
@@ -680,7 +678,7 @@ bool disk_quotas(const char *path, SMB_BIG_UINT *bsize, SMB_BIG_UINT *dfree, SMB
        DEBUG(5,("disk_quotas: looking for quotas file \"%s\"\n", name));
        if((file=sys_open(name, O_RDONLY,0))<0) {
                unbecome_root();
-               return(False);
+               return false;
        }
        command.op = Q_GETQUOTA;
        command.uid = euser_id;
@@ -695,7 +693,8 @@ bool disk_quotas(const char *path, SMB_BIG_UINT *bsize, SMB_BIG_UINT *dfree, SMB
        unbecome_root();
 
        if (ret < 0) {
-               DEBUG(5,("disk_quotas ioctl (Solaris) failed. Error = %s\n", strerror(errno) ));
+               DEBUG(5,("disk_quotas ioctl (Solaris) failed. Error = %s\n",
+                                       strerror(errno) ));
 
 #if defined(SUNOS5) && defined(VXFS_QUOTA)
                /* If normal quotactl() fails, try vxfs private calls */
@@ -703,24 +702,27 @@ bool disk_quotas(const char *path, SMB_BIG_UINT *bsize, SMB_BIG_UINT *dfree, SMB
                DEBUG(5,("disk_quotas: mount type \"%s\"\n", mnt.mnt_fstype));
                if ( 0 == strcmp ( mnt.mnt_fstype, "vxfs" )) {
                        bool retval;
-                       retval = disk_quotas_vxfs(name, path, bsize, dfree, dsize);
-                       return(retval);
+                       retval = disk_quotas_vxfs(name, path,
+                                       bsize, dfree, dsize);
+                       return retval;
                }
 #else
-               return(False);
+               return false;
 #endif
        }
 
        /* If softlimit is zero, set it equal to hardlimit.
         */
-  
-       if (D.dqb_bsoftlimit==0)
+
+       if (D.dqb_bsoftlimit==0) {
                D.dqb_bsoftlimit = D.dqb_bhardlimit;
+       }
 
-       /* Use softlimit to determine disk space. A user exceeding the quota is told
-        * that there's no space left. Writes might actually work for a bit if the
-        * hardlimit is set higher than softlimit. Effectively the disk becomes
-        * made of rubber latex and begins to expand to accommodate the user :-)
+       /* Use softlimit to determine disk space. A user exceeding the quota
+        * is told that there's no space left. Writes might actually work for
+        * a bit if the hardlimit is set higher than softlimit. Effectively
+        * the disk becomes made of rubber latex and begins to expand to
+        * accommodate the user :-)
         */
 
        if (D.dqb_bsoftlimit==0)
@@ -731,13 +733,15 @@ bool disk_quotas(const char *path, SMB_BIG_UINT *bsize, SMB_BIG_UINT *dfree, SMB
        if (D.dqb_curblocks > D.dqb_bsoftlimit) {
                *dfree = 0;
                *dsize = D.dqb_curblocks;
-       } else
+       } else {
                *dfree = D.dqb_bsoftlimit - D.dqb_curblocks;
-      
-       DEBUG(5,("disk_quotas for path \"%s\" returning  bsize %.0f, dfree %.0f, dsize %.0f\n",
+       }
+
+       DEBUG(5,("disk_quotas for path \"%s\" returning "
+               "bsize %.0f, dfree %.0f, dsize %.0f\n",
                path,(double)*bsize,(double)*dfree,(double)*dsize));
 
-       return(True);
+       return true;
 }
 
 
@@ -748,7 +752,7 @@ bool disk_quotas(const char *path, SMB_BIG_UINT *bsize, SMB_BIG_UINT *dfree, SMB
 try to get the disk space from disk quotas - OSF1 version
 ****************************************************************************/
 
-bool disk_quotas(const char *path, SMB_BIG_UINT *bsize, SMB_BIG_UINT *dfree, SMB_BIG_UINT *dsize)
+bool disk_quotas(const char *path, uint64_t *bsize, uint64_t *dfree, uint64_t *dsize)
 {
   int r, save_errno;
   struct dqblk D;
@@ -814,7 +818,7 @@ try to get the disk space from disk quotas (IRIX 6.2 version)
 #include <sys/quota.h>
 #include <mntent.h>
 
-bool disk_quotas(const char *path, SMB_BIG_UINT *bsize, SMB_BIG_UINT *dfree, SMB_BIG_UINT *dsize)
+bool disk_quotas(const char *path, uint64_t *bsize, uint64_t *dfree, uint64_t *dsize)
 {
   uid_t euser_id;
   int r;
@@ -832,7 +836,7 @@ bool disk_quotas(const char *path, SMB_BIG_UINT *bsize, SMB_BIG_UINT *dfree, SMB
     return(False) ;
   }
 
-  devno = S.st_dev ;
+  devno = S.st_ex_dev ;
   
   fp = setmntent(MOUNTED,"r");
   found = False ;
@@ -840,7 +844,7 @@ bool disk_quotas(const char *path, SMB_BIG_UINT *bsize, SMB_BIG_UINT *dfree, SMB
   while ((mnt = getmntent(fp))) {
     if ( sys_stat(mnt->mnt_dir,&S) == -1 )
       continue ;
-    if (S.st_dev == devno) {
+    if (S.st_ex_dev == devno) {
       found = True ;
       break ;
     }
@@ -966,8 +970,6 @@ bool disk_quotas(const char *path, SMB_BIG_UINT *bsize, SMB_BIG_UINT *dfree, SMB
 #endif
 #include <rpc/xdr.h>
 
-static int quotastat;
-
 static int my_xdr_getquota_args(XDR *xdrsp, struct getquota_args *args)
 {
        if (!xdr_string(xdrsp, &args->gqa_pathp, RQ_PATHLEN ))
@@ -979,10 +981,14 @@ static int my_xdr_getquota_args(XDR *xdrsp, struct getquota_args *args)
 
 static int my_xdr_getquota_rslt(XDR *xdrsp, struct getquota_rslt *gqr)
 {
+       int quotastat;
+
        if (!xdr_int(xdrsp, &quotastat)) {
                DEBUG(6,("nfs_quotas: Status bad or zero\n"));
                return 0;
        }
+       gqr->status = quotastat;
+
        if (!xdr_int(xdrsp, &gqr->getquota_rslt_u.gqr_rquota.rq_bsize)) {
                DEBUG(6,("nfs_quotas: Block size bad or zero\n"));
                return 0;
@@ -1007,7 +1013,7 @@ static int my_xdr_getquota_rslt(XDR *xdrsp, struct getquota_rslt *gqr)
 }
 
 /* Works on FreeBSD, too. :-) */
-static bool nfs_quotas(char *nfspath, uid_t euser_id, SMB_BIG_UINT *bsize, SMB_BIG_UINT *dfree, SMB_BIG_UINT *dsize)
+static bool nfs_quotas(char *nfspath, uid_t euser_id, uint64_t *bsize, uint64_t *dfree, uint64_t *dsize)
 {
        uid_t uid = euser_id;
        struct dqblk D;
@@ -1021,7 +1027,7 @@ static bool nfs_quotas(char *nfspath, uid_t euser_id, SMB_BIG_UINT *bsize, SMB_B
        enum clnt_stat clnt_stat;
        bool ret = True;
 
-       *bsize = *dfree = *dsize = (SMB_BIG_UINT)0;
+       *bsize = *dfree = *dsize = (uint64_t)0;
 
        len=strcspn(mnttype, ":");
        pathname=strstr(mnttype, ":");
@@ -1056,14 +1062,14 @@ static bool nfs_quotas(char *nfspath, uid_t euser_id, SMB_BIG_UINT *bsize, SMB_B
        }
 
        /* 
-        * quotastat returns 0 if the rpc call fails, 1 if quotas exist, 2 if there is
+        * gqr->status returns 0 if the rpc call fails, 1 if quotas exist, 2 if there is
         * no quota set, and 3 if no permission to get the quota.  If 0 or 3 return
         * something sensible.
         */   
 
-       switch ( quotastat ) {
+       switch (gqr.status) {
        case 0:
-               DEBUG(9,("nfs_quotas: Remote Quotas Failed!  Error \"%i\" \n", quotastat ));
+               DEBUG(9,("nfs_quotas: Remote Quotas Failed!  Error \"%i\" \n", gqr.status));
                ret = False;
                goto out;
 
@@ -1078,16 +1084,16 @@ static bool nfs_quotas(char *nfspath, uid_t euser_id, SMB_BIG_UINT *bsize, SMB_B
        case 3:
                D.dqb_bsoftlimit = 1;
                D.dqb_curblocks = 1;
-               DEBUG(9,("nfs_quotas: Remote Quotas returned \"%i\" \n", quotastat ));
+               DEBUG(9,("nfs_quotas: Remote Quotas returned \"%i\" \n", gqr.status));
                break;
 
        default:
-               DEBUG(9,("nfs_quotas: Remote Quotas Questionable!  Error \"%i\" \n", quotastat ));
+               DEBUG(9,("nfs_quotas: Remote Quotas Questionable!  Error \"%i\" \n", gqr.status));
                break;
        }
 
        DEBUG(10,("nfs_quotas: Let`s look at D a bit closer... status \"%i\" bsize \"%i\" active? \"%i\" bhard \"%i\" bsoft \"%i\" curb \"%i\" \n",
-                       quotastat,
+                       gqr.status,
                        gqr.getquota_rslt_u.gqr_rquota.rq_bsize,
                        gqr.getquota_rslt_u.gqr_rquota.rq_active,
                        gqr.getquota_rslt_u.gqr_rquota.rq_bhardlimit,
@@ -1102,7 +1108,7 @@ static bool nfs_quotas(char *nfspath, uid_t euser_id, SMB_BIG_UINT *bsize, SMB_B
        *bsize = gqr.getquota_rslt_u.gqr_rquota.rq_bsize;
        *dsize = D.dqb_bsoftlimit;
 
-       if (D.dqb_curblocks == D.dqb_curblocks == 1)
+       if (D.dqb_curblocks == 1)
                *bsize = DEV_BSIZE;
 
        if (D.dqb_curblocks > D.dqb_bsoftlimit) {
@@ -1132,7 +1138,7 @@ static bool nfs_quotas(char *nfspath, uid_t euser_id, SMB_BIG_UINT *bsize, SMB_B
 try to get the disk space from disk quotas - default version
 ****************************************************************************/
 
-bool disk_quotas(const char *path, SMB_BIG_UINT *bsize, SMB_BIG_UINT *dfree, SMB_BIG_UINT *dsize)
+bool disk_quotas(const char *path, uint64_t *bsize, uint64_t *dfree, uint64_t *dsize)
 {
   int r;
   struct dqblk D;
@@ -1148,9 +1154,9 @@ bool disk_quotas(const char *path, SMB_BIG_UINT *bsize, SMB_BIG_UINT *dfree, SMB
    * to have a significant performance boost when
    * lstat calls on /dev access this function.
    */
-  if ((sys_stat(path, &S)<0) || (devnm(S_IFBLK, S.st_dev, dev_disk, 256, 1)<0))
+  if ((sys_stat(path, &S)<0) || (devnm(S_IFBLK, S.st_ex_dev, dev_disk, 256, 1)<0))
 #else
-  if ((sys_stat(path, &S)<0) || (devnm(S_IFBLK, S.st_dev, dev_disk, 256, 0)<0)) 
+  if ((sys_stat(path, &S)<0) || (devnm(S_IFBLK, S.st_ex_dev, dev_disk, 256, 0)<0))
        return (False);
 #endif /* ifdef HPUX */
 
@@ -1179,7 +1185,7 @@ bool disk_quotas(const char *path, SMB_BIG_UINT *bsize, SMB_BIG_UINT *dfree, SMB
     
     if (sys_stat(path,&st) < 0)
         return False;
-    devno = st.st_dev;
+    devno = st.st_ex_dev;
 
     mntsize = getmntinfo(&mnts,MNT_NOWAIT);
     if (mntsize <= 0)
@@ -1188,7 +1194,7 @@ bool disk_quotas(const char *path, SMB_BIG_UINT *bsize, SMB_BIG_UINT *dfree, SMB
     for (i = 0; i < mntsize; i++) {
         if (sys_stat(mnts[i].f_mntonname,&st) < 0)
             return False;
-        if (st.st_dev == devno)
+        if (st.st_ex_dev == devno)
             break;
     }
     if (i == mntsize)
@@ -1351,14 +1357,14 @@ Hints for porting:
 #include <sys/fs/vx_aioctl.h>
 #include <sys/fs/vx_ioctl.h>
 
-bool disk_quotas_vxfs(const pstring name, char *path, SMB_BIG_UINT *bsize, SMB_BIG_UINT *dfree, SMB_BIG_UINT *dsize)
+bool disk_quotas_vxfs(const char *name, char *path, uint64_t *bsize, uint64_t *dfree, uint64_t *dsize)
 {
   uid_t user_id, euser_id;
   int ret;
   struct vx_dqblk D;
   struct vx_quotctl quotabuf;
   struct vx_genioctl genbuf;
-  pstring qfname;
+  char *qfname;
   int file;
 
   /*
@@ -1367,7 +1373,10 @@ bool disk_quotas_vxfs(const pstring name, char *path, SMB_BIG_UINT *bsize, SMB_B
    * it might be easier to examine and adjust it here.
    * Fortunately, VxFS seems not to mind at present.
    */
-  pstrcpy(qfname, name) ;
+  qfname = talloc_strdup(talloc_tos(), name);
+  if (!qfname) {
+         return false;
+  }
   /* pstrcat(qfname, "/quotas") ; */   /* possibly examine and adjust "name" */
 
   euser_id = geteuid();
@@ -1432,25 +1441,25 @@ bool disk_quotas_vxfs(const pstring name, char *path, SMB_BIG_UINT *bsize, SMB_B
 
 #else /* WITH_QUOTAS */
 
-bool disk_quotas(const char *path,SMB_BIG_UINT *bsize,SMB_BIG_UINT *dfree,SMB_BIG_UINT *dsize)
+bool disk_quotas(const char *path,uint64_t *bsize,uint64_t *dfree,uint64_t *dsize)
 {
-  (*bsize) = 512; /* This value should be ignored */
+       (*bsize) = 512; /* This value should be ignored */
 
-  /* And just to be sure we set some values that hopefully */
-  /* will be larger that any possible real-world value     */
-  (*dfree) = (SMB_BIG_UINT)-1;
-  (*dsize) = (SMB_BIG_UINT)-1;
+       /* And just to be sure we set some values that hopefully */
+       /* will be larger that any possible real-world value     */
+       (*dfree) = (uint64_t)-1;
+       (*dsize) = (uint64_t)-1;
 
-  /* As we have select not to use quotas, allways fail */
-  return False;
+       /* As we have select not to use quotas, allways fail */
+       return false;
 }
 #endif /* WITH_QUOTAS */
 
 #else /* HAVE_SYS_QUOTAS */
-/* wrapper to the new sys_quota interface 
+/* wrapper to the new sys_quota interface
    this file should be removed later
    */
-bool disk_quotas(const char *path,SMB_BIG_UINT *bsize,SMB_BIG_UINT *dfree,SMB_BIG_UINT *dsize)
+bool disk_quotas(const char *path,uint64_t *bsize,uint64_t *dfree,uint64_t *dsize)
 {
        int r;
        SMB_DISK_QUOTA D;