Changeset 6ad454f in mainline for uspace/srv/vfs/vfs_ops.c

Timestamp:

2017-12-08T21:03:35Z (8 years ago)

Author:

Jiří Zárevúcky <zarevucky.jiri@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

a8c7a6d

Parents:

9246016

git-author:

Jiří Zárevúcky <zarevucky.jiri@…> (2017-12-08 06:01:16)

git-committer:

Jiří Zárevúcky <zarevucky.jiri@…> (2017-12-08 21:03:35)

Message:

Pass file handles separately from error codes.

File:

• uspace/srv/vfs/vfs_ops.c (modified) (8 diffs)

uspace/srv/vfs/vfs_ops.c

@@ -104 +104 @@
                 /* Assign the old file to newfd. */
                 rc = vfs_fd_assign(oldfile, newfd);
+                *out_fd = newfd;
         } else {
                 vfs_file_t *newfile;
-                int newfd = vfs_fd_alloc(&newfile, desc);
-                if (newfd >= 0) {
+                rc = vfs_fd_alloc(&newfile, desc, out_fd);
+                if (rc == EOK) {
                         newfile->node = oldfile->node;
                         newfile->permissions = oldfile->permissions;
@@ -114 +115 @@
                         vfs_file_put(newfile);
                 }
-                rc = newfd;
         }
         vfs_file_put(oldfile);
         
-        if (rc < 0) {
-                return rc;
-        }
-        
-        *out_fd = rc;
-        return EOK;
+        return rc;
 }
 
@@ -228 +223 @@
 
 int vfs_op_mount(int mpfd, unsigned service_id, unsigned flags,
-    unsigned instance, const char *opts, const char *fs_name, int *outfd)
+    unsigned instance, const char *opts, const char *fs_name, int *out_fd)
 {
         int rc;
         vfs_file_t *mp = NULL;
         vfs_file_t *file = NULL;
-        int fd = -1;
+        *out_fd = -1;
         
         if (!(flags & VFS_MOUNT_CONNECT_ONLY)) {
@@ -259 +254 @@
         
         if (!(flags & VFS_MOUNT_NO_REF)) {
-                fd = vfs_fd_alloc(&file, false);
-                if (fd < 0) {
-                        rc = fd;
+                rc = vfs_fd_alloc(&file, false, out_fd);
+                if (rc != EOK) {
                         goto out;
                 }
@@ -300 +294 @@
                 vfs_file_put(file);
 
-        if (rc != EOK && fd >= 0) {
-                vfs_fd_free(fd);
-                fd = 0;
-        }
-        
-        *outfd = fd;
+        if (rc != EOK && *out_fd >= 0) {
+                vfs_fd_free(*out_fd);
+                *out_fd = -1;
+        }
+        
         return rc;
 }
@@ -833 +826 @@
 }
 
-int vfs_op_wait_handle(bool high_fd)
-{
-        return vfs_wait_handle_internal(high_fd);
+int vfs_op_wait_handle(bool high_fd, int *out_fd)
+{
+        return vfs_wait_handle_internal(high_fd, out_fd);
 }
 
@@ -897 +890 @@
         
         vfs_file_t *file;
-        int fd = vfs_fd_alloc(&file, false);
-        if (fd < 0) {
+        rc = vfs_fd_alloc(&file, false, out_fd);
+        if (rc != EOK) {
                 vfs_node_put(node);
                 vfs_file_put(parent);
-                return fd;
+                return rc;
         }
         assert(file != NULL);
@@ -915 +908 @@
         fibril_rwlock_read_unlock(&namespace_rwlock);
 
-        *out_fd = fd;
         return EOK;
 }