Remove the double-close() warning
authorBarret Rhoden <brho@cs.berkeley.edu>
Tue, 22 Mar 2016 21:45:49 +0000 (17:45 -0400)
committerBarret Rhoden <brho@cs.berkeley.edu>
Thu, 31 Mar 2016 20:53:42 +0000 (16:53 -0400)
It's legal to attempt to close a bad FD.  We had a nasty bug that this
caught a long time ago, but now we have programs that do this all the time
(ssh).

Since I fixed the shutdown() problem, we no longer have any known-buggy
programs that are double-closing.

Signed-off-by: Barret Rhoden <brho@cs.berkeley.edu>
kern/src/syscall.c

index f09893e..17d8dcf 100644 (file)
@@ -1628,11 +1628,6 @@ static intreg_t sys_close(struct proc *p, int fd)
        }
        /* 9ns, should also handle errors (bad FD, etc) */
        retval = sysclose(fd);
-       if (retval < 0) {
-               /* no one checks their retvals.  a double close will cause problems. */
-               printk("[kernel] sys_close failed: proc %d fd %d.  Check your rets.\n",
-                      p->pid, fd);
-       }
        return retval;
 }