Minor changes, to file perms and unimplemented functions

blobfuse/directoryapis.cpp

@@ -61,7 +61,7 @@ int azs_readdir(const char *path, void *buf, fuse_fill_dir_t filler, off_t, stru
 		    	if (dir_ent->d_type == DT_DIR)
 		    	{
 		            struct stat stbuf;
-		            stbuf.st_mode = S_IFDIR | 0700;
+		            stbuf.st_mode = S_IFDIR | 0770;
 		            stbuf.st_nlink = 2;
 	 				stbuf.st_size = 4096;
 	 	            filler(buf, dir_ent->d_name, &stbuf, 0);
@@ -72,7 +72,7 @@ int azs_readdir(const char *path, void *buf, fuse_fill_dir_t filler, off_t, stru
 		    		stat((mntPathString + dir_ent->d_name).c_str(), &buffer);
 
 		            struct stat stbuf;
-		            stbuf.st_mode = S_IFREG | 0700; // Regular file (not a directory)
+		            stbuf.st_mode = S_IFREG | 0770; // Regular file (not a directory)
 		            stbuf.st_nlink = 1;
 		            stbuf.st_size = buffer.st_size;
 		            filler(buf, dir_ent->d_name, &stbuf, 0); // TODO: Add stat information.  Consider FUSE_FILL_DIR_PLUS.
@@ -152,7 +152,7 @@ int azs_readdir(const char *path, void *buf, fuse_fill_dir_t filler, off_t, stru
 	                if ((prev_token_str.size() > 0) && (strcmp(prev_token_str.c_str(), directorySignifier.c_str()) != 0))
 	                {
 	                    struct stat stbuf;
-	                    stbuf.st_mode = S_IFREG | 0700; // Regular file (not a directory)
+	                    stbuf.st_mode = S_IFREG | 0770; // Regular file (not a directory)
 	                    stbuf.st_nlink = 1;
 	                    stbuf.st_size = listResults[i].content_length;
 	                    fillerResult = filler(buf, prev_token_str.c_str(), &stbuf, 0); // TODO: Add stat information.  Consider FUSE_FILL_DIR_PLUS.
@@ -168,7 +168,7 @@ int azs_readdir(const char *path, void *buf, fuse_fill_dir_t filler, off_t, stru
 	                if (prev_token_str.size() > 0)
 	                {
 	                    struct stat stbuf;
-	                    stbuf.st_mode = S_IFDIR | 0700;
+	                    stbuf.st_mode = S_IFDIR | 0770;
 	                    stbuf.st_nlink = 2;
 	                    fillerResult = filler(buf, prev_token_str.c_str(), &stbuf, 0);
 	                    if (AZS_PRINT)

blobfuse/fileapis.cpp

@@ -188,7 +188,7 @@ int azs_open(const char *path, struct fuse_file_info *fi)
     }
 
     // TODO: Actual access control
-    fchmod(res, 0700);
+    fchmod(res, 0770);
 
     // Stora the open file handle, and whether or not the file should be uploaded on close().
     // TODO: Optimize the scenario where the file is open for read/write, but no actual writing occurs, to not upload the blob.
@@ -270,7 +270,7 @@ int azs_create(const char *path, mode_t mode, struct fuse_file_info *fi)
         }
         if (access(mntPathCopy, F_OK) != 0)
         {
-            mkdir(mntPathCopy, 0700);
+            mkdir(mntPathCopy, 0770);
         }
         *cur = '/';
         cur = cur + 1;

blobfuse/utilities.cpp

@@ -40,7 +40,7 @@ int ensure_files_directory_exists(const std::string file_path)
             struct stat st;
             if (stat(copypath, &st) != 0)
             {
-                status = mkdir(copypath, 0700);
+                status = mkdir(copypath, 0770);
             }
             *slash = '/';
         }
@@ -167,7 +167,7 @@ int azs_getattr(const char *path, struct stat *stbuf)
     // If we're at the root, we know it's a directory
     if (strlen(path) == 1)
     {
-        stbuf->st_mode = S_IFDIR | 0700; // TODO: proper access control.
+        stbuf->st_mode = S_IFDIR | 0770; // TODO: proper access control.
         stbuf->st_nlink = 2; // Directories should have a hard-link count of 2 + (# child directories).  We don't have that count, though, so we jsut use 2 for now.  TODO: Evaluate if we could keep this accurate or not.
         stbuf->st_size = 4096;
         return 0;
@@ -206,7 +206,7 @@ int azs_getattr(const char *path, struct stat *stbuf)
         {
             fprintf(stdout, "Blob found!  Name = %s\n", path);
         }
-        stbuf->st_mode = S_IFREG | 0700; // Regular file (not a directory)
+        stbuf->st_mode = S_IFREG | 0770; // Regular file (not a directory)
         stbuf->st_nlink = 1;
         stbuf->st_size = blob_property.size;
         return 0;
@@ -233,7 +233,7 @@ int azs_getattr(const char *path, struct stat *stbuf)
             {
                 fprintf(stdout, "Directory %s found with return value: %d!\n", blobNameStr.c_str(), dirSize);
             }
-            stbuf->st_mode = S_IFDIR | 0700;
+            stbuf->st_mode = S_IFDIR | 0770;
             // If st_nlink = 2, means direcotry is empty.
             // Direcotry size will affect behaviour for mv, rmdir, cp etc.
             stbuf->st_nlink = dirSize == D_EMPTY ? 2 : 3;
@@ -289,7 +289,7 @@ int azs_access(const char * /*path*/, int /*mask*/)
 
 int azs_readlink(const char * /*path*/, char * /*buf*/, size_t /*size*/)
 {
-    return 1; // ignore for mow
+    return -EINVAL; // not a symlink
 }
 
 int azs_fsync(const char * /*path*/, int /*isdatasync*/, struct fuse_file_info * /*fi*/)
@@ -300,12 +300,14 @@ int azs_fsync(const char * /*path*/, int /*isdatasync*/, struct fuse_file_info *
 int azs_chown(const char * /*path*/, uid_t /*uid*/, gid_t /*gid*/)
 {
     //TODO: Implement
+//    return -ENOSYS;
     return 0;
 }
 
 int azs_chmod(const char * /*path*/, mode_t /*mode*/)
 {
     //TODO: Implement
+//    return -ENOSYS;
     return 0;
 
 }
@@ -314,6 +316,7 @@ int azs_chmod(const char * /*path*/, mode_t /*mode*/)
 int azs_utimens(const char * /*path*/, const struct timespec [2] /*ts[2]*/)
 {
     //TODO: Implement
+//    return -ENOSYS;
     return 0;
 }
 //  #endif
@@ -608,17 +611,17 @@ int azs_rename(const char *src, const char *dst)
 
 int azs_setxattr(const char * /*path*/, const char * /*name*/, const char * /*value*/, size_t /*size*/, int /*flags*/)
 {
-    return 0;
+    return -ENOSYS;
 }
 int azs_getxattr(const char * /*path*/, const char * /*name*/, char * /*value*/, size_t /*size*/)
 {
-    return 0;
+    return -ENOSYS;
 }
 int azs_listxattr(const char * /*path*/, char * /*list*/, size_t /*size*/)
 {
-    return 0;
+    return -ENOSYS;
 }
 int azs_removexattr(const char * /*path*/, const char * /*name*/)
 {
-    return 0;
+    return -ENOSYS;
 }