crtxcr/cgitsb
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

cache.c: make all io-related functions return errno on error

Browse Source 
We'll need proper return-values from these functions to make the cache
behave correctly (which includes giving proper error messages).

Noticed-by: Jim Meyering <jim@meyering.net>
Signed-off-by: Lars Hjemli <hjemli@gmail.com>
This commit is contained in:
Lars Hjemli 2008-05-18 23:16:50 +02:00
parent 6102bcfce4
commit d402811bd2
1 changed files with 24 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

33
cache.c
View File

@ -66,12 +66,16 @@ static int open_slot(struct cache_slot *slot)
} }
/* Close the active cache slot */ /* Close the active cache slot */
static void close_slot(struct cache_slot *slot) static int close_slot(struct cache_slot *slot)
{ {
int err = 0;
if (slot->cache_fd > 0) { if (slot->cache_fd > 0) {
close(slot->cache_fd); if (close(slot->cache_fd))
slot->cache_fd = -1; err = errno;
else
slot->cache_fd = -1;
} }
return err;
} }
/* Print the content of the active cache slot (but skip the key). */ /* Print the content of the active cache slot (but skip the key). */
@ -116,12 +120,16 @@ static int is_modified(struct cache_slot *slot)
} }
/* Close an open lockfile */ /* Close an open lockfile */
static void close_lock(struct cache_slot *slot) static int close_lock(struct cache_slot *slot)
{ {
int err = 0;
if (slot->lock_fd > 0) { if (slot->lock_fd > 0) {
close(slot->lock_fd); if (close(slot->lock_fd))
slot->lock_fd = -1; err = errno;
else
slot->lock_fd = -1;
} }
return err;
} }
/* Create a lockfile used to store the generated content for a cache /* Create a lockfile used to store the generated content for a cache
@ -134,7 +142,8 @@ static int lock_slot(struct cache_slot *slot)
S_IRUSR|S_IWUSR); S_IRUSR|S_IWUSR);
if (slot->lock_fd == -1) if (slot->lock_fd == -1)
return errno; return errno;
write(slot->lock_fd, slot->key, slot->keylen + 1); if (write(slot->lock_fd, slot->key, slot->keylen + 1) < 0)
return errno;
return 0; return 0;
} }
@ -150,7 +159,11 @@ static int unlock_slot(struct cache_slot *slot, int replace_old_slot)
err = rename(slot->lock_name, slot->cache_name); err = rename(slot->lock_name, slot->cache_name);
else else
err = unlink(slot->lock_name); err = unlink(slot->lock_name);
return err;
if (err)
return errno;
return 0;
} }
/* Generate the content for the current cache slot by redirecting /* Generate the content for the current cache slot by redirecting
@ -177,7 +190,9 @@ static int fill_slot(struct cache_slot *slot)
return errno; return errno;
/* Close the temporary filedescriptor */ /* Close the temporary filedescriptor */
close(tmp); if (close(tmp))
return errno;
return 0; return 0;
} }