simplify

diff --git a/file.h b/file.h
index 99f0122a6cb4f37d8925a2ae4606b847d8bd1cde..1d56145674dc0c1c5c5d249539ba06aa2738f1bc 100644 (file)
--- a/file.h
+++ b/file.h
@@ -1,5 +1,5 @@
 struct file {
-  enum { FD_CLOSED, FD_NONE, FD_PIPE, FD_INODE } type;
+  enum { FD_NONE, FD_PIPE, FD_INODE } type;
   int ref; // reference count
   char readable;
   char writable;

diff --git a/pipe.c b/pipe.c
index 8a580f95e63e5c536174fc9d06cf22aab13e3d16..7f18e98ba5300ed6252de148362a19ea54db4d1c 100644 (file)
--- a/pipe.c
+++ b/pipe.c
@@ -47,14 +47,10 @@ pipealloc(struct file **f0, struct file **f1)
  bad:
   if(p)
     kfree((char*)p, PAGE);
-  if(*f0){
-    (*f0)->type = FD_NONE;
+  if(*f0)
     fileclose(*f0);
-  }
-  if(*f1){
-    (*f1)->type = FD_NONE;
+  if(*f1)
     fileclose(*f1);
-  }
   return -1;
 }
 

diff --git a/sysfile.c b/sysfile.c
index 8808073bcbd23e8df19b954ca4c920e0877064c3..efc42b00c1e7b925ff81612d2d728db82043c704 100644 (file)
--- a/sysfile.c
+++ b/sysfile.c
@@ -127,17 +127,17 @@ sys_link(void)
   iunlock(ip);
 
   if((dp = nameiparent(new, name)) == 0)
-    goto  bad;
+    goto bad;
   ilock(dp);
-  if(dp->dev != ip->dev || dirlink(dp, name, ip->inum) < 0)
+  if(dp->dev != ip->dev || dirlink(dp, name, ip->inum) < 0){
+    iunlockput(dp);
     goto bad;
+  }
   iunlockput(dp);
   iput(ip);
   return 0;
 
 bad:
-  if(dp)
-    iunlockput(dp);
   ilock(ip);
   ip->nlink--;
   iupdate(ip);
@@ -212,7 +212,7 @@ sys_unlink(void)
 }
 
 static struct inode*
-create(char *path, int canexist, short type, short major, short minor)
+create(char *path, short type, short major, short minor)
 {
   uint off;
   struct inode *ip, *dp;
@@ -222,10 +222,10 @@ create(char *path, int canexist, short type, short major, short minor)
     return 0;
   ilock(dp);
 
-  if(canexist && (ip = dirlookup(dp, name, &off)) != 0){
+  if((ip = dirlookup(dp, name, &off)) != 0){
     iunlockput(dp);
     ilock(ip);
-    if(ip->type != type || ip->major != major || ip->minor != minor){
+    if(ip->type != type || type != T_FILE){
       iunlockput(ip);
       return 0;
     }
@@ -250,17 +250,8 @@ create(char *path, int canexist, short type, short major, short minor)
       panic("create dots");
   }
 
-  if(dirlink(dp, name, ip->inum) < 0){
-    if(type == T_DIR){
-      dp->nlink--;
-      iupdate(dp);
-    }
-    iunlockput(dp);
-
-    ip->nlink = 0;
-    iunlockput(ip);
-    return 0;
-  }
+  if(dirlink(dp, name, ip->inum) < 0)
+    panic("create: dirlink");
 
   iunlockput(dp);
   return ip;
@@ -278,13 +269,13 @@ sys_open(void)
     return -1;
 
   if(omode & O_CREATE){
-    if((ip = create(path, 1, T_FILE, 0, 0)) == 0)
+    if((ip = create(path, T_FILE, 0, 0)) == 0)
       return -1;
   } else {
     if((ip = namei(path)) == 0)
       return -1;
     ilock(ip);
-    if(ip->type == T_DIR && (omode & (O_RDWR|O_WRONLY))){
+    if(ip->type == T_DIR && omode != O_RDONLY){
       iunlockput(ip);
       return -1;
     }
@@ -318,7 +309,7 @@ sys_mknod(void)
   if((len=argstr(0, &path)) < 0 ||
      argint(1, &major) < 0 ||
      argint(2, &minor) < 0 ||
-     (ip = create(path, 0, T_DEV, major, minor)) == 0)
+     (ip = create(path, T_DEV, major, minor)) == 0)
     return -1;
   iunlockput(ip);
   return 0;
@@ -330,7 +321,7 @@ sys_mkdir(void)
   char *path;
   struct inode *ip;
 
-  if(argstr(0, &path) < 0 || (ip = create(path, 0, T_DIR, 0, 0)) == 0)
+  if(argstr(0, &path) < 0 || (ip = create(path, T_DIR, 0, 0)) == 0)
     return -1;
   iunlockput(ip);
   return 0;