Changes in uspace/lib/posix/source/unistd.c [5759642f:6afc9d7] in mainline

File:

• uspace/lib/posix/source/unistd.c (modified) (16 diffs)

uspace/lib/posix/source/unistd.c

@@ -105 +105 @@
 char *posix_getcwd(char *buf, size_t size)
 {
-        /* Native getcwd() does not set any errno despite the fact that general
-         * usage pattern of this function depends on it (caller is repeatedly
-         * guessing the required size of the buffer by checking for ERANGE on
-         * failure). */
-        if (size == 0) {
-                errno = EINVAL;
+        char *p = getcwd(buf, size);
+
+        if (p == NULL) {
+                errno = -errno;
                 return NULL;
         }
-        
-        /* Save the original value to comply with the "no modification on
-         * success" semantics.
-         */
-        int orig_errno = errno;
-        errno = EOK;
-        
-        char *ret = getcwd(buf, size);
-        if (ret == NULL) {
-                /* Check errno to avoid shadowing other possible errors. */
-                if (errno == EOK) {
-                        errno = ERANGE;
-                }
-        } else {
-                /* Success, restore previous errno value. */
-                errno = orig_errno;
-        }
-        
-        return ret;
+
+        return p;
 }
 
@@ -141 +122 @@
 int posix_chdir(const char *path)
 {
-        return errnify(chdir, path);
+        return negerrno(chdir, path);
 }
 
@@ -194 +175 @@
 int posix_close(int fildes)
 {
-        return errnify(close, fildes);
+        return negerrno(close, fildes);
 }
 
@@ -207 +188 @@
 ssize_t posix_read(int fildes, void *buf, size_t nbyte)
 {
-        return errnify(read, fildes, buf, nbyte);
+        return negerrno(read, fildes, buf, nbyte);
 }
 
@@ -220 +201 @@
 ssize_t posix_write(int fildes, const void *buf, size_t nbyte)
 {
-        return errnify(write, fildes, buf, nbyte);
+        return negerrno(write, fildes, buf, nbyte);
+}
+
+/**
+ * Reposition read/write file offset
+ *
+ * @param fildes File descriptor of the opened file.
+ * @param offset New offset in the file.
+ * @param whence The position from which the offset argument is specified.
+ * @return Upon successful completion, returns the resulting offset
+ *         as measured in bytes from the beginning of the file, -1 otherwise.
+ */
+posix_off_t posix_lseek(int fildes, posix_off_t offset, int whence)
+{
+        return negerrno(lseek, fildes, offset, whence);
 }
 
@@ -231 +226 @@
 int posix_fsync(int fildes)
 {
-        return errnify(fsync, fildes);
+        return negerrno(fsync, fildes);
 }
 
@@ -243 +238 @@
 int posix_ftruncate(int fildes, posix_off_t length)
 {
-        return errnify(ftruncate, fildes, (aoff64_t) length);
+        return negerrno(ftruncate, fildes, (aoff64_t) length);
 }
 
@@ -254 +249 @@
 int posix_rmdir(const char *path)
 {
-        return errnify(rmdir, path);
+        return negerrno(rmdir, path);
 }
 
@@ -265 +260 @@
 int posix_unlink(const char *path)
 {
-        return errnify(unlink, path);
+        return negerrno(unlink, path);
 }
 
@@ -289 +284 @@
 int posix_dup2(int fildes, int fildes2)
 {
-        return errnify(dup2, fildes, fildes2);
+        return negerrno(dup2, fildes, fildes2);
 }
 
@@ -307 +302 @@
                  * Check file existence by attempting to open it.
                  */
-                int fd = open(path, O_RDONLY);
+                int fd = negerrno(open, path, O_RDONLY);
                 if (fd < 0) {
-                        errno = -fd;
+                        /* errno was set by open() */
                         return -1;
                 }
@@ -375 +370 @@
         // TODO: low priority, just a compile-time dependency of binutils
         not_implemented();
+        return -1;
 }
 
@@ -385 +381 @@
         // TODO: low priority, just a compile-time dependency of binutils
         not_implemented();
+        return -1;
 }
 
@@ -397 +394 @@
         // TODO: low priority, just a compile-time dependency of binutils
         not_implemented();
+        return -1;
 }
 
@@ -409 +407 @@
         // TODO: low priority, just a compile-time dependency of binutils
         not_implemented();
+        return -1;
 }
 
@@ -420 +419 @@
         // TODO: low priority, just a compile-time dependency of binutils
         not_implemented();
+        return -1;
 }