1 files changed, 138 insertions, 149 deletions

diff --git a/arch/um/os-Linux/file.c b/arch/um/os-Linux/file.c
index f83462758627..b5afcfd0f861 100644
--- a/arch/um/os-Linux/file.c
+++ b/arch/um/os-Linux/file.c
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2002 Jeff Dike (jdike@karaya.com)
+ * Copyright (C) 2002 - 2007 Jeff Dike (jdike@{addtoit,linux.intel}.com)
  * Licensed under the GPL
  */
 
@@ -8,18 +8,16 @@
 #include <errno.h>
 #include <fcntl.h>
 #include <signal.h>
-#include <sys/types.h>
-#include <sys/stat.h>
-#include <sys/socket.h>
-#include <sys/un.h>
 #include <sys/ioctl.h>
 #include <sys/mount.h>
-#include <sys/uio.h>
+#include <sys/socket.h>
+#include <sys/stat.h>
+#include <sys/un.h>
+#include "kern_constants.h"
 #include "os.h"
 #include "user.h"
-#include "kern_util.h"
 
-static void copy_stat(struct uml_stat *dst, struct stat64 *src)
+static void copy_stat(struct uml_stat *dst, const struct stat64 *src)
 {
         *dst = ((struct uml_stat) {
                 .ust_dev     = src->st_dev,     /* device */
@@ -43,10 +41,10 @@ int os_stat_fd(const int fd, struct uml_stat *ubuf)
         int err;
 
         CATCH_EINTR(err = fstat64(fd, &sbuf));
-        if(err < 0)
+        if (err < 0)
                 return -errno;
 
-        if(ubuf != NULL)
+        if (ubuf != NULL)
                 copy_stat(ubuf, &sbuf);
         return err;
 }
@@ -56,27 +54,26 @@ int os_stat_file(const char *file_name, struct uml_stat *ubuf)
         struct stat64 sbuf;
         int err;
 
-        do {
-                err = stat64(file_name, &sbuf);
-        } while((err < 0) && (errno == EINTR)) ;
-
-        if(err < 0)
+        CATCH_EINTR(err = stat64(file_name, &sbuf));
+        if (err < 0)
                 return -errno;
 
-        if(ubuf != NULL)
+        if (ubuf != NULL)
                 copy_stat(ubuf, &sbuf);
         return err;
 }
 
-int os_access(const char* file, int mode)
+int os_access(const char *file, int mode)
 {
         int amode, err;
 
-        amode=(mode&OS_ACC_R_OK ? R_OK : 0) | (mode&OS_ACC_W_OK ? W_OK : 0) |
-              (mode&OS_ACC_X_OK ? X_OK : 0) | (mode&OS_ACC_F_OK ? F_OK : 0) ;
+        amode = (mode & OS_ACC_R_OK ? R_OK : 0) |
+                (mode & OS_ACC_W_OK ? W_OK : 0) |
+                (mode & OS_ACC_X_OK ? X_OK : 0) |
+                (mode & OS_ACC_F_OK ? F_OK : 0);
 
         err = access(file, amode);
-        if(err < 0)
+        if (err < 0)
                 return -errno;
 
         return 0;
@@ -88,7 +85,7 @@ int os_ioctl_generic(int fd, unsigned int cmd, unsigned long arg)
         int err;
 
         err = ioctl(fd, cmd, arg);
-        if(err < 0)
+        if (err < 0)
                 return -errno;
 
         return err;
@@ -97,7 +94,7 @@ int os_ioctl_generic(int fd, unsigned int cmd, unsigned long arg)
 /* FIXME: ensure namebuf in os_get_if_name is big enough */
 int os_get_ifname(int fd, char* namebuf)
 {
-        if(ioctl(fd, SIOCGIFNAME, namebuf) < 0)
+        if (ioctl(fd, SIOCGIFNAME, namebuf) < 0)
                 return -errno;
 
         return 0;
@@ -108,37 +105,22 @@ int os_set_slip(int fd)
         int disc, sencap;
 
         disc = N_SLIP;
-        if(ioctl(fd, TIOCSETD, &disc) < 0)
+        if (ioctl(fd, TIOCSETD, &disc) < 0)
                 return -errno;
 
         sencap = 0;
-        if(ioctl(fd, SIOCSIFENCAP, &sencap) < 0)
+        if (ioctl(fd, SIOCSIFENCAP, &sencap) < 0)
                 return -errno;
 
         return 0;
 }
 
-int os_set_owner(int fd, int pid)
-{
-        if(fcntl(fd, F_SETOWN, pid) < 0){
-                int save_errno = errno;
-
-                if(fcntl(fd, F_GETOWN, 0) != pid)
-                        return -save_errno;
-        }
-
-        return 0;
-}
-
 int os_mode_fd(int fd, int mode)
 {
         int err;
 
-        do {
-                err = fchmod(fd, mode);
-        } while((err < 0) && (errno==EINTR)) ;
-
-        if(err < 0)
+        CATCH_EINTR(err = fchmod(fd, mode));
+        if (err < 0)
                 return -errno;
 
         return 0;
@@ -150,64 +132,73 @@ int os_file_type(char *file)
         int err;
 
         err = os_stat_file(file, &buf);
-        if(err < 0)
+        if (err < 0)
                 return err;
 
-        if(S_ISDIR(buf.ust_mode))
+        if (S_ISDIR(buf.ust_mode))
                 return OS_TYPE_DIR;
-        else if(S_ISLNK(buf.ust_mode))
+        else if (S_ISLNK(buf.ust_mode))
                 return OS_TYPE_SYMLINK;
-        else if(S_ISCHR(buf.ust_mode))
+        else if (S_ISCHR(buf.ust_mode))
                 return OS_TYPE_CHARDEV;
-        else if(S_ISBLK(buf.ust_mode))
+        else if (S_ISBLK(buf.ust_mode))
                 return OS_TYPE_BLOCKDEV;
-        else if(S_ISFIFO(buf.ust_mode))
+        else if (S_ISFIFO(buf.ust_mode))
                 return OS_TYPE_FIFO;
-        else if(S_ISSOCK(buf.ust_mode))
+        else if (S_ISSOCK(buf.ust_mode))
                 return OS_TYPE_SOCK;
         else return OS_TYPE_FILE;
 }
 
-int os_file_mode(char *file, struct openflags *mode_out)
+int os_file_mode(const char *file, struct openflags *mode_out)
 {
         int err;
 
         *mode_out = OPENFLAGS();
 
         err = access(file, W_OK);
-        if(err && (errno != EACCES))
+        if (err && (errno != EACCES))
                 return -errno;
-        else if(!err)
+        else if (!err)
                 *mode_out = of_write(*mode_out);
 
         err = access(file, R_OK);
-        if(err && (errno != EACCES))
+        if (err && (errno != EACCES))
                 return -errno;
-        else if(!err)
+        else if (!err)
                 *mode_out = of_read(*mode_out);
 
         return err;
 }
 
-int os_open_file(char *file, struct openflags flags, int mode)
+int os_open_file(const char *file, struct openflags flags, int mode)
 {
         int fd, err, f = 0;
 
-        if(flags.r && flags.w) f = O_RDWR;
-        else if(flags.r) f = O_RDONLY;
-        else if(flags.w) f = O_WRONLY;
+        if (flags.r && flags.w)
+                f = O_RDWR;
+        else if (flags.r)
+                f = O_RDONLY;
+        else if (flags.w)
+                f = O_WRONLY;
         else f = 0;
 
-        if(flags.s) f |= O_SYNC;
-        if(flags.c) f |= O_CREAT;
-        if(flags.t) f |= O_TRUNC;
-        if(flags.e) f |= O_EXCL;
+        if (flags.s)
+                f |= O_SYNC;
+        if (flags.c)
+                f |= O_CREAT;
+        if (flags.t)
+                f |= O_TRUNC;
+        if (flags.e)
+                f |= O_EXCL;
+        if (flags.a)
+                f |= O_APPEND;
 
         fd = open64(file, f, mode);
-        if(fd < 0)
+        if (fd < 0)
                 return -errno;
 
-        if(flags.cl && fcntl(fd, F_SETFD, 1)){
+        if (flags.cl && fcntl(fd, F_SETFD, 1)) {
                 err = -errno;
                 close(fd);
                 return err;
@@ -216,7 +207,7 @@ int os_open_file(char *file, struct openflags flags, int mode)
         return fd;
 }
 
-int os_connect_socket(char *name)
+int os_connect_socket(const char *name)
 {
         struct sockaddr_un sock;
         int fd, err;
@@ -225,13 +216,13 @@ int os_connect_socket(char *name)
         snprintf(sock.sun_path, sizeof(sock.sun_path), "%s", name);
 
         fd = socket(AF_UNIX, SOCK_STREAM, 0);
-        if(fd < 0) {
+        if (fd < 0) {
                 err = -errno;
                 goto out;
         }
 
         err = connect(fd, (struct sockaddr *) &sock, sizeof(sock));
-        if(err) {
+        if (err) {
                 err = -errno;
                 goto out_close;
         }
@@ -254,7 +245,7 @@ int os_seek_file(int fd, unsigned long long offset)
         unsigned long long actual;
 
         actual = lseek64(fd, offset, SEEK_SET);
-        if(actual != offset)
+        if (actual != offset)
                 return -errno;
         return 0;
 }
@@ -263,7 +254,7 @@ int os_read_file(int fd, void *buf, int len)
 {
         int n = read(fd, buf, len);
 
-        if(n < 0)
+        if (n < 0)
                 return -errno;
         return n;
 }
@@ -272,37 +263,38 @@ int os_write_file(int fd, const void *buf, int len)
 {
         int n = write(fd, (void *) buf, len);
 
-        if(n < 0)
+        if (n < 0)
                 return -errno;
         return n;
 }
 
-int os_file_size(char *file, unsigned long long *size_out)
+int os_file_size(const char *file, unsigned long long *size_out)
 {
         struct uml_stat buf;
         int err;
 
         err = os_stat_file(file, &buf);
-        if(err < 0){
-                printk("Couldn't stat \"%s\" : err = %d\n", file, -err);
+        if (err < 0) {
+                printk(UM_KERN_ERR "Couldn't stat \"%s\" : err = %d\n", file,
+                       -err);
                 return err;
         }
 
-        if(S_ISBLK(buf.ust_mode)){
+        if (S_ISBLK(buf.ust_mode)) {
                 int fd;
                 long blocks;
 
                 fd = open(file, O_RDONLY, 0);
-                if(fd < 0) {
+                if (fd < 0) {
                         err = -errno;
-                        printk("Couldn't open \"%s\", errno = %d\n", file,
-                               errno);
+                        printk(UM_KERN_ERR "Couldn't open \"%s\", "
+                               "errno = %d\n", file, errno);
                         return err;
                 }
-                if(ioctl(fd, BLKGETSIZE, &blocks) < 0){
+                if (ioctl(fd, BLKGETSIZE, &blocks) < 0) {
                         err = -errno;
-                        printk("Couldn't get the block size of \"%s\", "
-                               "errno = %d\n", file, errno);
+                        printk(UM_KERN_ERR "Couldn't get the block size of "
+                               "\"%s\", errno = %d\n", file, errno);
                         close(fd);
                         return err;
                 }
@@ -314,14 +306,15 @@ int os_file_size(char *file, unsigned long long *size_out)
         return 0;
 }
 
-int os_file_modtime(char *file, unsigned long *modtime)
+int os_file_modtime(const char *file, unsigned long *modtime)
 {
         struct uml_stat buf;
         int err;
 
         err = os_stat_file(file, &buf);
-        if(err < 0){
-                printk("Couldn't stat \"%s\" : err = %d\n", file, -err);
+        if (err < 0) {
+                printk(UM_KERN_ERR "Couldn't stat \"%s\" : err = %d\n", file,
+                       -err);
                 return err;
         }
 
@@ -329,26 +322,13 @@ int os_file_modtime(char *file, unsigned long *modtime)
         return 0;
 }
 
-int os_get_exec_close(int fd, int *close_on_exec)
-{
-        int ret;
-
-        CATCH_EINTR(ret = fcntl(fd, F_GETFD));
-
-        if(ret < 0)
-                return -errno;
-
-        *close_on_exec = (ret & FD_CLOEXEC) ? 1 : 0;
-        return ret;
-}
-
 int os_set_exec_close(int fd)
 {
         int err;
 
         CATCH_EINTR(err = fcntl(fd, F_SETFD, FD_CLOEXEC));
 
-        if(err < 0)
+        if (err < 0)
                 return -errno;
         return err;
 }
@@ -358,53 +338,51 @@ int os_pipe(int *fds, int stream, int close_on_exec)
         int err, type = stream ? SOCK_STREAM : SOCK_DGRAM;
 
         err = socketpair(AF_UNIX, type, 0, fds);
-        if(err < 0)
+        if (err < 0)
                 return -errno;
 
-        if(!close_on_exec)
+        if (!close_on_exec)
                 return 0;
 
         err = os_set_exec_close(fds[0]);
-        if(err < 0)
+        if (err < 0)
                 goto error;
 
         err = os_set_exec_close(fds[1]);
-        if(err < 0)
+        if (err < 0)
                 goto error;
 
         return 0;
 
  error:
-        printk("os_pipe : Setting FD_CLOEXEC failed, err = %d\n", -err);
+        printk(UM_KERN_ERR "os_pipe : Setting FD_CLOEXEC failed, err = %d\n",
+               -err);
         close(fds[1]);
         close(fds[0]);
         return err;
 }
 
-int os_set_fd_async(int fd, int owner)
+int os_set_fd_async(int fd)
 {
-        int err;
+        int err, flags;
+
+        flags = fcntl(fd, F_GETFL);
+        if (flags < 0)
+                return -errno;
 
-        /* XXX This should do F_GETFL first */
-        if(fcntl(fd, F_SETFL, O_ASYNC | O_NONBLOCK) < 0){
+        flags |= O_ASYNC | O_NONBLOCK;
+        if (fcntl(fd, F_SETFL, flags) < 0) {
                 err = -errno;
-                printk("os_set_fd_async : failed to set O_ASYNC and "
-                       "O_NONBLOCK on fd # %d, errno = %d\n", fd, errno);
+                printk(UM_KERN_ERR "os_set_fd_async : failed to set O_ASYNC "
+                       "and O_NONBLOCK on fd # %d, errno = %d\n", fd, errno);
                 return err;
         }
-#ifdef notdef
-        if(fcntl(fd, F_SETFD, 1) < 0){
-                printk("os_set_fd_async : Setting FD_CLOEXEC failed, "
-                       "errno = %d\n", errno);
-        }
-#endif
 
-        if((fcntl(fd, F_SETSIG, SIGIO) < 0) ||
-           (fcntl(fd, F_SETOWN, owner) < 0)){
+        if ((fcntl(fd, F_SETSIG, SIGIO) < 0) ||
+            (fcntl(fd, F_SETOWN, os_getpid()) < 0)) {
                 err = -errno;
-                printk("os_set_fd_async : Failed to fcntl F_SETOWN "
-                       "(or F_SETSIG) fd %d to pid %d, errno = %d\n", fd,
-                       owner, errno);
+                printk(UM_KERN_ERR "os_set_fd_async : Failed to fcntl F_SETOWN "
+                       "(or F_SETSIG) fd %d, errno = %d\n", fd, errno);
                 return err;
         }
 
@@ -413,10 +391,14 @@ int os_set_fd_async(int fd, int owner)
 
 int os_clear_fd_async(int fd)
 {
-        int flags = fcntl(fd, F_GETFL);
+        int flags;
+
+        flags = fcntl(fd, F_GETFL);
+        if (flags < 0)
+                return -errno;
 
         flags &= ~(O_ASYNC | O_NONBLOCK);
-        if(fcntl(fd, F_SETFL, flags) < 0)
+        if (fcntl(fd, F_SETFL, flags) < 0)
                 return -errno;
         return 0;
 }
@@ -426,11 +408,15 @@ int os_set_fd_block(int fd, int blocking)
         int flags;
 
         flags = fcntl(fd, F_GETFL);
+        if (flags < 0)
+                return -errno;
 
-        if(blocking) flags &= ~O_NONBLOCK;
-        else flags |= O_NONBLOCK;
+        if (blocking)
+                flags &= ~O_NONBLOCK;
+        else
+                flags |= O_NONBLOCK;
 
-        if(fcntl(fd, F_SETFL, flags) < 0)
+        if (fcntl(fd, F_SETFL, flags) < 0)
                 return -errno;
 
         return 0;
@@ -441,7 +427,7 @@ int os_accept_connection(int fd)
         int new;
 
         new = accept(fd, NULL, 0);
-        if(new < 0)
+        if (new < 0)
                 return -errno;
         return new;
 }
@@ -462,15 +448,17 @@ int os_shutdown_socket(int fd, int r, int w)
 {
         int what, err;
 
-        if(r && w) what = SHUT_RDWR;
-        else if(r) what = SHUT_RD;
-        else if(w) what = SHUT_WR;
-        else {
-                printk("os_shutdown_socket : neither r or w was set\n");
+        if (r && w)
+                what = SHUT_RDWR;
+        else if (r)
+                what = SHUT_RD;
+        else if (w)
+                what = SHUT_WR;
+        else
                 return -EINVAL;
-        }
+
         err = shutdown(fd, what);
-        if(err < 0)
+        if (err < 0)
                 return -errno;
         return 0;
 }
@@ -494,19 +482,20 @@ int os_rcv_fd(int fd, int *helper_pid_out)
         msg.msg_flags = 0;
 
         n = recvmsg(fd, &msg, 0);
-        if(n < 0)
+        if (n < 0)
                 return -errno;
-        else if(n != iov.iov_len)
+        else if (n != iov.iov_len)
                 *helper_pid_out = -1;
 
         cmsg = CMSG_FIRSTHDR(&msg);
-        if(cmsg == NULL){
-                printk("rcv_fd didn't receive anything, error = %d\n", errno);
+        if (cmsg == NULL) {
+                printk(UM_KERN_ERR "rcv_fd didn't receive anything, "
+                       "error = %d\n", errno);
                 return -1;
         }
-        if((cmsg->cmsg_level != SOL_SOCKET) ||
-           (cmsg->cmsg_type != SCM_RIGHTS)){
-                printk("rcv_fd didn't receive a descriptor\n");
+        if ((cmsg->cmsg_level != SOL_SOCKET) ||
+            (cmsg->cmsg_type != SCM_RIGHTS)) {
+                printk(UM_KERN_ERR "rcv_fd didn't receive a descriptor\n");
                 return -1;
         }
 
@@ -514,29 +503,28 @@ int os_rcv_fd(int fd, int *helper_pid_out)
         return new;
 }
 
-int os_create_unix_socket(char *file, int len, int close_on_exec)
+int os_create_unix_socket(const char *file, int len, int close_on_exec)
 {
         struct sockaddr_un addr;
         int sock, err;
 
         sock = socket(PF_UNIX, SOCK_DGRAM, 0);
-        if(sock < 0)
+        if (sock < 0)
                 return -errno;
 
-        if(close_on_exec) {
+        if (close_on_exec) {
                 err = os_set_exec_close(sock);
-                if(err < 0)
-                        printk("create_unix_socket : close_on_exec failed, "
-                       "err = %d", -err);
+                if (err < 0)
+                        printk(UM_KERN_ERR "create_unix_socket : "
+                               "close_on_exec failed, err = %d", -err);
         }
 
         addr.sun_family = AF_UNIX;
 
-        /* XXX Be more careful about overflow */
         snprintf(addr.sun_path, len, "%s", file);
 
         err = bind(sock, (struct sockaddr *) &addr, sizeof(addr));
-        if(err < 0)
+        if (err < 0)
                 return -errno;
 
         return sock;
@@ -557,17 +545,18 @@ int os_lock_file(int fd, int excl)
         int err, save;
 
         err = fcntl(fd, F_SETLK, &lock);
-        if(!err)
+        if (!err)
                 goto out;
 
         save = -errno;
         err = fcntl(fd, F_GETLK, &lock);
-        if(err){
+        if (err) {
                 err = -errno;
                 goto out;
         }
 
-        printk("F_SETLK failed, file already locked by pid %d\n", lock.l_pid);
+        printk(UM_KERN_ERR "F_SETLK failed, file already locked by pid %d\n",
+               lock.l_pid);
         err = save;
  out:
         return err;