Move a couple of functions from osl.c to fd.c.
authorAndre Noll <maan@systemlinux.org>
Thu, 5 Jun 2008 19:14:22 +0000 (21:14 +0200)
committerAndre Noll <maan@systemlinux.org>
Thu, 5 Jun 2008 19:14:22 +0000 (21:14 +0200)
This makes them available also to fsck (coming soon).

errlist
fd.c
fd.h
osl.c
osl_core.h

diff --git a/errlist b/errlist
index fb24427..eb7f282 100644 (file)
--- a/errlist
+++ b/errlist
@@ -24,7 +24,6 @@ BAD_ROW_NUM                   "invalid row number"
 INDEX_CORRUPTION               "index corruption detected"
 INVALID_OBJECT                 "reference to invalid object"
 STAT                           "can not stat file"
-WRITE                          "write error"
 LSEEK                          "lseek error"
 BUSY                           "table is busy"
 SHORT_TABLE                    "table too short"
diff --git a/fd.c b/fd.c
index 8487760..b889d9d 100644 (file)
--- a/fd.c
+++ b/fd.c
 #include "osl.h"
 #include "error.h"
 
+/**
+ * Wrapper for the write system call.
+ *
+ * \param fd The file descriptor to write to.
+ * \param buf The buffer to write.
+ * \param size The length of \a buf in bytes.
+ *
+ * This function writes out the given buffer and retries if an interrupt
+ * occurred during the write.
+ *
+ * \return On success, the number of bytes written is returned, otherwise, a
+ * negative error code is returned.
+ *
+ * \sa write(2).
+ */
+static ssize_t __write(int fd, const void *buf, size_t size)
+{
+       ssize_t ret;
+
+       for (;;) {
+               ret = write(fd, buf, size);
+               if ((ret < 0) && (errno == EAGAIN || errno == EINTR))
+                       continue;
+               return ret >= 0? ret : -ERRNO_TO_ERROR(errno);
+       }
+}
+
 /**
  * Write a buffer to a file descriptor, re-write on short writes.
  *
@@ -33,9 +60,9 @@ int write_all(int fd, const char *buf, size_t *len)
        assert(total);
        *len = 0;
        while (*len < total) {
-               int ret = write(fd, buf + *len, total - *len);
-               if (ret == -1)
-                       return -ERRNO_TO_ERROR(errno);
+               int ret = __write(fd, buf + *len, total - *len);
+               if (ret < 0)
+                       return ret;
                *len += ret;
        }
        return 1;
@@ -64,6 +91,32 @@ int osl_open(const char *path, int flags, mode_t mode)
        return -ERRNO_TO_ERROR(errno);
 }
 
+/**
+ * Open a file, write the given buffer and close the file.
+ *
+ * \param filename Full path to the file to open.
+ * \param buf The buffer to write to the file.
+ * \param size The size of \a buf.
+ *
+ * \return Standard.
+ */
+int write_file(const char *filename, const void *buf, size_t size)
+{
+       int ret, fd;
+
+       ret = osl_open(filename, O_WRONLY | O_CREAT | O_EXCL, 0644);
+       if (ret < 0)
+               return ret;
+       fd = ret;
+       ret = write_all(fd, buf, &size);
+       if (ret < 0)
+               goto out;
+       ret = 1;
+out:
+       close(fd);
+       return ret;
+}
+
 /**
  * Wrapper for chdir(2).
  *
@@ -222,3 +275,50 @@ int osl_munmap(void *start, size_t length)
                strerror(err));
        return -ERRNO_TO_ERROR(err);
 }
+
+/**
+ * Allocate a sufficiently large string and print into it.
+ *
+ * \param fmt A usual format string.
+ *
+ * Produce output according to \p fmt. No artificial bound on the length of the
+ * resulting string is imposed.
+ *
+ * \return This function either returns a pointer to a string that must be
+ * freed by the caller or \p NULL if memory allocation failed.
+ *
+ * \sa printf(3).
+ */
+__must_check __printf_1_2 __malloc char *make_message(const char *fmt, ...)
+{
+       int n;
+       size_t size = 100;
+       char *p = malloc(size);
+
+       if (!p)
+               return NULL;
+       while (1) {
+               char *q;
+               va_list ap;
+               /* Try to print in the allocated space. */
+               va_start(ap, fmt);
+               n = vsnprintf(p, size, fmt, ap);
+               va_end(ap);
+               /* If that worked, return the string. */
+               if (n > -1 && n < size)
+                       break;
+               /* Else try again with more space. */
+               if (n > -1) /* glibc 2.1 */
+                       size = n + 1; /* precisely what is needed */
+               else /* glibc 2.0 */
+                       size *= 2; /* twice the old size */
+               q = realloc(p, size);
+               if (!q) {
+                       free(p);
+                       return NULL;
+               }
+               p = q;
+       }
+       return p;
+}
+
diff --git a/fd.h b/fd.h
index 0ca272d..4b80206 100644 (file)
--- a/fd.h
+++ b/fd.h
@@ -12,6 +12,8 @@ int para_fchdir(int fd);
 int mmap_full_file(const char *filename, int open_mode, void **map,
        size_t *size, int *fd_ptr);
 int osl_munmap(void *start, size_t length);
+int write_all(int fd, const char *buf, size_t *len);
+int write_file(const char *filename, const void *buf, size_t size);
 
 /**
  * A wrapper for mkdir(2).
diff --git a/osl.c b/osl.c
index 0b3e88c..f0506a5 100644 (file)
--- a/osl.c
+++ b/osl.c
@@ -8,7 +8,6 @@
 #include <dirent.h> /* readdir() */
 #include <assert.h>
 
-
 #include "log.h"
 #include "osl.h"
 #include "error.h"
 #include "list.h"
 #include "osl_core.h"
 
-/**
- * Allocate a sufficiently large string and print into it.
- *
- * \param fmt A usual format string.
- *
- * Produce output according to \p fmt. No artificial bound on the length of the
- * resulting string is imposed.
- *
- * \return This function either returns a pointer to a string that must be
- * freed by the caller or \p NULL if memory allocation failed.
- *
- * \sa printf(3).
- */
-static __must_check __printf_1_2 __malloc char *make_message(const char *fmt, ...)
-{
-       int n;
-       size_t size = 100;
-       char *p = malloc(size);
-
-       if (!p)
-               return NULL;
-       while (1) {
-               char *q;
-               va_list ap;
-               /* Try to print in the allocated space. */
-               va_start(ap, fmt);
-               n = vsnprintf(p, size, fmt, ap);
-               va_end(ap);
-               /* If that worked, return the string. */
-               if (n > -1 && n < size)
-                       break;
-               /* Else try again with more space. */
-               if (n > -1) /* glibc 2.1 */
-                       size = n + 1; /* precisely what is needed */
-               else /* glibc 2.0 */
-                       size *= 2; /* twice the old size */
-               q = realloc(p, size);
-               if (!q) {
-                       free(p);
-                       return NULL;
-               }
-               p = q;
-       }
-       return p;
-}
-
 /* Taken from Drepper: How to write shared libraries, Appendix B. */
 #include <stddef.h>
 #define MSGSTRFIELD(line) MSGSTRFIELD1(line)
@@ -154,88 +107,6 @@ static int __lseek(int fd, off_t *offset, int whence)
        return 1;
 }
 
-/**
- * Wrapper for the write system call.
- *
- * \param fd The file descriptor to write to.
- * \param buf The buffer to write.
- * \param size The length of \a buf in bytes.
- *
- * This function writes out the given buffer and retries if an interrupt
- * occurred during the write.
- *
- * \return On success, the number of bytes written is returned, otherwise, the
- * function returns \p -E_OSL_WRITE.
- *
- * \sa write(2).
- */
-static ssize_t __write(int fd, const void *buf, size_t size)
-{
-       ssize_t ret;
-
-       for (;;) {
-               ret = write(fd, buf, size);
-               if ((ret < 0) && (errno == EAGAIN || errno == EINTR))
-                       continue;
-               return ret >= 0? ret : -E_OSL_WRITE;
-       }
-}
-
-/**
- * Write the whole buffer to a file descriptor.
- *
- * \param fd The file descriptor to write to.
- * \param buf The buffer to write.
- * \param size The length of \a buf in bytes.
- *
- * This function writes the given buffer and continues on short writes and
- * when interrupted by a signal.
- *
- * \return Positive on success, negative on errors. Possible errors: any
- * errors returned by para_write().
- *
- * \sa para_write().
- */
-static ssize_t write_all(int fd, const void *buf, size_t size)
-{
-//     DEBUG_LOG("writing %zu bytes\n", size);
-       const char *b = buf;
-       while (size) {
-               ssize_t ret = __write(fd, b, size);
-//             DEBUG_LOG("ret: %zd\n", ret);
-               if (ret < 0)
-                       return ret;
-               b += ret;
-               size -= ret;
-       }
-       return 1;
-}
-/**
- * Open a file, write the given buffer and close the file.
- *
- * \param filename Full path to the file to open.
- * \param buf The buffer to write to the file.
- * \param size The size of \a buf.
- *
- * \return Standard.
- */
-static int write_file(const char *filename, const void *buf, size_t size)
-{
-       int ret, fd;
-
-       ret = osl_open(filename, O_WRONLY | O_CREAT | O_EXCL, 0644);
-       if (ret < 0)
-               return ret;
-       fd = ret;
-       ret = write_all(fd, buf, size);
-       if (ret < 0)
-               goto out;
-       ret = 1;
-out:
-       close(fd);
-       return ret;
-}
-
 static int append_file(const char *filename, char *header, size_t header_size,
        char *data, size_t data_size, uint32_t *new_pos)
 {
@@ -247,11 +118,11 @@ static int append_file(const char *filename, char *header, size_t header_size,
                return ret;
        fd = ret;
        if (header && header_size) {
-               ret = write_all(fd, header, header_size);
+               ret = write_all(fd, header, &header_size);
                if (ret < 0)
                        goto out;
        }
-       ret = write_all(fd, data, data_size);
+       ret = write_all(fd, data, &data_size);
        if (ret < 0)
                goto out;
        if (new_pos) {
index 858743a..966ef6f 100644 (file)
@@ -9,7 +9,7 @@
 #include "rbtree.h"
 #include "hash.h"
 
-static __must_check __printf_1_2 __malloc char *make_message(const char *fmt, ...);
+__must_check __printf_1_2 __malloc char *make_message(const char *fmt, ...);
 
 /** Internal representation of a column of an osl table. */
 struct osl_column {