+* Fri Jun 10 2005 Andrew McNab <Andrew.McNab@man.ac.uk>
+- Tidy up gsexec vs GridSiteDiskPerms
+* Fri Jun 10 2005 Andrew McNab <Andrew.McNab@man.ac.uk>
+- ==== GridSite version 1.1.10 ====
 * Wed Jun 8 2005 Andrew McNab <Andrew.McNab@man.ac.uk>
 - Add GridSiteDiskMode Apache directive to set file
   permissions.
 
 %attr(-, root, root) %(echo ${MYPREFIX:-/usr})/share/man/man1/htmkdir.1.gz
 
 %files gsexec
-%attr(-, root, root) %(echo ${MYPREFIX:-/usr})/sbin/gsexec
+%attr(4510, root, apache) %(echo ${MYPREFIX:-/usr})/sbin/gsexec
 %attr(-, root, root) %(echo ${MYPREFIX:-/usr})/share/man/man8/gsexec.8.gz
 
     if (diskmode_env != NULL)
       {
         diskmode_apr = 0;
-        sscanf(diskmode_env, "%d", &diskmode_apr);
+        sscanf(diskmode_env, "%i", &diskmode_apr);
       
         diskmode_t = S_IRUSR | S_IWUSR;
         
         if (diskmode_apr & APR_WREAD ) diskmode_t |= S_IROTH;
         
         diskmode_t &= (S_IRUSR | S_IWUSR | S_IRGRP | S_IWGRP | S_IROTH);
-        
+
+// log_err("diskmode_env=%s diskmode_apr=%x diskmode_t=%o ~diskmode_t=%o\n", diskmode_env, diskmode_apr, diskmode_t, ~diskmode_t);  
+              
         umask(~diskmode_t);
       }
 #ifdef AP_SUEXEC_UMASK
     else umask(~(S_IRUSR | S_IWUSR));
 #endif /* AP_SUEXEC_UMASK */
     
-    
-
     /* 
      * Be sure to close the log file so the CGI can't
      * mess with it.  If the exec fails, it will be reopened