summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTimo Teräs <timo.teras@iki.fi>2015-04-13 09:41:49 +0300
committerTimo Teräs <timo.teras@iki.fi>2015-04-13 09:42:27 +0300
commit3490ff789a9375d3047c6580f24a55157ebef4f0 (patch)
treea07390ce63f550a0f8e8f0736df5079354ee6602
parent43955329325c823579ded743efb324cc8ea1daba (diff)
downloadapk-tools-3490ff789a9375d3047c6580f24a55157ebef4f0.tar.gz
apk-tools-3490ff789a9375d3047c6580f24a55157ebef4f0.tar.bz2
apk-tools-3490ff789a9375d3047c6580f24a55157ebef4f0.tar.xz
apk-tools-3490ff789a9375d3047c6580f24a55157ebef4f0.zip
fix tee io error handling
use ERR_PTR mechanism, and handle it at all places.
-rw-r--r--src/database.c2
-rw-r--r--src/io.c8
2 files changed, 6 insertions, 4 deletions
diff --git a/src/database.c b/src/database.c
index b71c9df..364b4df 100644
--- a/src/database.c
+++ b/src/database.c
@@ -2586,7 +2586,7 @@ static int apk_db_unpack_pkg(struct apk_database *db,
apk_blob_push_blob(&b, tmpprefix);
apk_pkg_format_cache_pkg(b, pkg);
cache_bs = apk_bstream_tee(bs, db->cache_fd, tmpcacheitem, NULL, NULL);
- if (cache_bs != NULL)
+ if (!IS_ERR_OR_NULL(cache_bs))
bs = cache_bs;
else
apk_warning(PKG_VER_FMT": unable to cache: %s",
diff --git a/src/io.c b/src/io.c
index 765afc5..fa8a048 100644
--- a/src/io.c
+++ b/src/io.c
@@ -421,22 +421,24 @@ static void tee_close(void *stream, size_t *size)
struct apk_bstream *apk_bstream_tee(struct apk_bstream *from, int atfd, const char *to, apk_progress_cb cb, void *cb_ctx)
{
struct apk_tee_bstream *tbs;
- int fd;
+ int fd, r;
if (IS_ERR_OR_NULL(from)) return ERR_CAST(from);
fd = openat(atfd, to, O_CREAT | O_RDWR | O_TRUNC | O_CLOEXEC,
S_IRUSR | S_IWUSR | S_IRGRP | S_IROTH);
if (fd < 0) {
+ r = errno;
from->close(from, NULL);
- return NULL;
+ return ERR_PTR(-r);
}
tbs = malloc(sizeof(struct apk_tee_bstream));
if (!tbs) {
+ r = errno;
close(fd);
from->close(from, NULL);
- return NULL;
+ return ERR_PTR(-r);
}
tbs->bs = (struct apk_bstream) {