Fixes insert_file()
[akaros.git] / kern / src / vfs.c
index e333612..bc43859 100644 (file)
@@ -2245,14 +2245,22 @@ int claim_fd(struct files_struct *open_files, int file_desc)
 
 /* Inserts the file in the files_struct, returning the corresponding new file
  * descriptor, or an error code.  We start looking for open fds from low_fd. */
-int insert_file(struct files_struct *open_files, struct file *file, int low_fd, int must)
+int insert_file(struct files_struct *open_files, struct file *file, int low_fd,
+                bool must)
 {
-       int slot;
+       int slot, ret;
        spin_lock(&open_files->lock);
-       if (must)
-               slot = __claim_fd(open_files, low_fd);
-       else
+       if (must) {
+               ret = __claim_fd(open_files, low_fd);
+               if (ret < 0) {
+                       spin_unlock(&open_files->lock);
+                       return ret;
+               }
+               assert(!ret);   /* issues with claim_fd returning status, not the fd */
+               slot = low_fd;
+       } else {
                slot = __get_fd(open_files, low_fd);
+       }
 
        if (slot < 0) {
                spin_unlock(&open_files->lock);