summaryrefslogtreecommitdiff
path: root/content
diff options
context:
space:
mode:
authorMichael Drake <tlsa@netsurf-browser.org>2014-05-24 15:19:01 +0100
committerMichael Drake <tlsa@netsurf-browser.org>2014-05-24 15:19:01 +0100
commitfb5af8a1b612e210384a1a6a4f8d5ee5fafef054 (patch)
treefcb9db3d54952bb8eea8e2ad6155b09a449513e8 /content
parentce4d52e72aa0fc09bd2207db175bc83c278bb0c6 (diff)
downloadnetsurf-fb5af8a1b612e210384a1a6a4f8d5ee5fafef054.tar.gz
netsurf-fb5af8a1b612e210384a1a6a4f8d5ee5fafef054.tar.bz2
Check return value from write() to avoid build warnings.
Diffstat (limited to 'content')
-rw-r--r--content/fs_backing_store.c16
1 files changed, 14 insertions, 2 deletions
diff --git a/content/fs_backing_store.c b/content/fs_backing_store.c
index 29856813f..b08a8e7f9 100644
--- a/content/fs_backing_store.c
+++ b/content/fs_backing_store.c
@@ -700,6 +700,7 @@ static nserror write_entries(struct store_state *state)
{
int fd;
char *fname = NULL;
+ ssize_t written;
nserror ret;
if (state->entries_dirty == false) {
@@ -718,8 +719,13 @@ static nserror write_entries(struct store_state *state)
return NSERROR_SAVE_FAILED;
}
- write(fd, state->entries, state->last_entry * sizeof(struct store_entry));
+ written = write(fd, state->entries,
+ state->last_entry * sizeof(struct store_entry));
close(fd);
+ if (written < 0) {
+ /* TODO: Delete the file? */
+ return NSERROR_SAVE_FAILED;
+ }
return NSERROR_OK;
}
@@ -1043,6 +1049,7 @@ store(nsurl *url,
{
nserror ret;
struct store_entry *bse;
+ ssize_t written;
int fd;
/* check backing store is initialised */
@@ -1064,10 +1071,15 @@ store(nsurl *url,
return NSERROR_SAVE_FAILED;
}
+
LOG(("Writing %d bytes from %p", datalen, data));
- write(fd, data, datalen);
+ written = write(fd, data, datalen);
close(fd);
+ if (written < 0) {
+ /* TODO: Delete the file? */
+ return NSERROR_SAVE_FAILED;
+ }
return NSERROR_OK;
}