fix tee io error handling

use ERR_PTR mechanism, and handle it at all places.
cute-signatures
Timo Teräs 2015-04-13 09:41:49 +03:00
parent 4395532932
commit 3490ff789a
2 changed files with 6 additions and 4 deletions

View File

@ -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",

View File

@ -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) {