From 1fdd452f47299be60cac9acbd8e7c864d9c174bd Mon Sep 17 00:00:00 2001 From: Elliott Hughes Date: Mon, 23 Mar 2015 13:33:57 -0700 Subject: Always use strerror to report errno in recovery. Change-Id: I7009959043150fabf5853a43ee2448c7fbea176e --- updater/blockimg.c | 39 ++++++++++++++++++++------------------- 1 file changed, 20 insertions(+), 19 deletions(-) (limited to 'updater') diff --git a/updater/blockimg.c b/updater/blockimg.c index 9c39634fc..90b55b2d2 100644 --- a/updater/blockimg.c +++ b/updater/blockimg.c @@ -464,7 +464,7 @@ static void EnumerateStash(const char* dirname, StashCallback callback, void* da if (directory == NULL) { if (errno != ENOENT) { - fprintf(stderr, "failed to opendir %s (errno %d)\n", dirname, errno); + fprintf(stderr, "opendir \"%s\" failed: %s\n", dirname, strerror(errno)); } return; } @@ -495,7 +495,7 @@ static void EnumerateStash(const char* dirname, StashCallback callback, void* da } if (closedir(directory) == -1) { - fprintf(stderr, "failed to closedir %s (errno %d)\n", dirname, errno); + fprintf(stderr, "closedir \"%s\" failed: %s\n", dirname, strerror(errno)); } } @@ -508,7 +508,7 @@ static void UpdateFileSize(const char* fn, void* data) { } if (stat(fn, &st) == -1) { - fprintf(stderr, "failed to stat %s (errno %d)\n", fn, errno); + fprintf(stderr, "stat \"%s\" failed: %s\n", fn, strerror(errno)); return; } @@ -524,7 +524,7 @@ static void DeleteFile(const char* fn, void* data) { fprintf(stderr, "deleting %s\n", fn); if (unlink(fn) == -1 && errno != ENOENT) { - fprintf(stderr, "failed to unlink %s (errno %d)\n", fn, errno); + fprintf(stderr, "unlink \"%s\" failed: %s\n", fn, strerror(errno)); } } } @@ -553,7 +553,7 @@ static void DeleteStash(const char* base) { if (rmdir(dirname) == -1) { if (errno != ENOENT && errno != ENOTDIR) { - fprintf(stderr, "failed to rmdir %s (errno %d)\n", dirname, errno); + fprintf(stderr, "rmdir \"%s\" failed: %s\n", dirname, strerror(errno)); } } @@ -587,7 +587,7 @@ static int LoadStash(const char* base, const char* id, int verify, int* blocks, if (res == -1) { if (errno != ENOENT || printnoent) { - fprintf(stderr, "failed to stat %s (errno %d)\n", fn, errno); + fprintf(stderr, "stat \"%s\" failed: %s\n", fn, strerror(errno)); } goto lsout; } @@ -602,7 +602,7 @@ static int LoadStash(const char* base, const char* id, int verify, int* blocks, fd = TEMP_FAILURE_RETRY(open(fn, O_RDONLY)); if (fd == -1) { - fprintf(stderr, "failed to open %s (errno %d)\n", fn, errno); + fprintf(stderr, "open \"%s\" failed: %s\n", fn, strerror(errno)); goto lsout; } @@ -663,7 +663,7 @@ static int WriteStash(const char* base, const char* id, int blocks, uint8_t* buf fd = TEMP_FAILURE_RETRY(open(fn, O_WRONLY | O_CREAT | O_TRUNC | O_SYNC, STASH_FILE_MODE)); if (fd == -1) { - fprintf(stderr, "failed to create %s (errno %d)\n", fn, errno); + fprintf(stderr, "failed to create \"%s\": %s\n", fn, strerror(errno)); goto wsout; } @@ -672,12 +672,12 @@ static int WriteStash(const char* base, const char* id, int blocks, uint8_t* buf } if (fsync(fd) == -1) { - fprintf(stderr, "failed to fsync %s (errno %d)\n", fn, errno); + fprintf(stderr, "fsync \"%s\" failed: %s\n", fn, strerror(errno)); goto wsout; } if (rename(fn, cn) == -1) { - fprintf(stderr, "failed to rename %s to %s (errno %d)\n", fn, cn, errno); + fprintf(stderr, "rename(\"%s\", \"%s\") failed: %s\n", fn, cn, strerror(errno)); goto wsout; } @@ -737,14 +737,14 @@ static int CreateStash(State* state, int maxblocks, const char* blockdev, char** res = stat(dirname, &st); if (res == -1 && errno != ENOENT) { - ErrorAbort(state, "failed to stat %s (errno %d)\n", dirname, errno); + ErrorAbort(state, "stat \"%s\" failed: %s\n", dirname, strerror(errno)); goto csout; } else if (res != 0) { fprintf(stderr, "creating stash %s\n", dirname); res = mkdir(dirname, STASH_DIRECTORY_MODE); if (res != 0) { - ErrorAbort(state, "failed to create %s (errno %d)\n", dirname, errno); + ErrorAbort(state, "mkdir \"%s\" failed: %s\n", dirname, strerror(errno)); goto csout; } @@ -1408,7 +1408,7 @@ static int PerformCommandErase(CommandParameters* params) { } if (fstat(params->fd, &st) == -1) { - fprintf(stderr, "failed to fstat device to erase (errno %d)\n", errno); + fprintf(stderr, "failed to fstat device to erase: %s\n", strerror(errno)); goto pceout; } @@ -1437,7 +1437,7 @@ static int PerformCommandErase(CommandParameters* params) { blocks[1] = (tgt->pos[i * 2 + 1] - tgt->pos[i * 2]) * (uint64_t) BLOCKSIZE; if (ioctl(params->fd, BLKDISCARD, &blocks) == -1) { - fprintf(stderr, "failed to blkdiscard (errno %d)\n", errno); + fprintf(stderr, "BLKDISCARD ioctl failed: %s\n", strerror(errno)); // Continue anyway, nothing we can do } } @@ -1574,7 +1574,7 @@ static Value* PerformBlockImageUpdate(const char* name, State* state, int argc, params.fd = TEMP_FAILURE_RETRY(open(blockdev_filename->data, O_RDWR)); if (params.fd == -1) { - fprintf(stderr, "failed to open %s: %s", blockdev_filename->data, strerror(errno)); + fprintf(stderr, "open \"%s\" failed: %s\n", blockdev_filename->data, strerror(errno)); goto pbiudone; } @@ -1587,8 +1587,9 @@ static Value* PerformBlockImageUpdate(const char* name, State* state, int argc, pthread_attr_init(&attr); pthread_attr_setdetachstate(&attr, PTHREAD_CREATE_JOINABLE); - if (pthread_create(¶ms.thread, &attr, unzip_new_data, ¶ms.nti) != 0) { - fprintf(stderr, "failed to create a thread (errno %d)\n", errno); + int error = pthread_create(¶ms.thread, &attr, unzip_new_data, ¶ms.nti); + if (error != 0) { + fprintf(stderr, "pthread_create failed: %s\n", strerror(error)); goto pbiudone; } } @@ -1719,7 +1720,7 @@ static Value* PerformBlockImageUpdate(const char* name, State* state, int argc, pbiudone: if (params.fd != -1) { if (fsync(params.fd) == -1) { - fprintf(stderr, "failed to fsync device (errno %d)\n", errno); + fprintf(stderr, "fsync failed: %s\n", strerror(errno)); } TEMP_FAILURE_RETRY(close(params.fd)); } @@ -1875,7 +1876,7 @@ Value* RangeSha1Fn(const char* name, State* state, int argc, Expr* argv[]) { int fd = open(blockdev_filename->data, O_RDWR); if (fd < 0) { - ErrorAbort(state, "failed to open %s: %s", blockdev_filename->data, strerror(errno)); + ErrorAbort(state, "open \"%s\" failed: %s", blockdev_filename->data, strerror(errno)); goto done; } -- cgit v1.2.3