summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>2013-05-02 22:51:50 -0400
committerZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>2013-05-02 22:55:38 -0400
commit11f22867f9cdbfc6656905861090cded91a41bd7 (patch)
tree584278caebffe696db13119ca8b2e67fc7d08410
parent44a6b1b68029833893f6e9cee35aa27a974038f6 (diff)
time-dst: use _cleanup_
-rw-r--r--src/shared/path-util.c3
-rw-r--r--src/shared/time-dst.c69
2 files changed, 32 insertions, 40 deletions
diff --git a/src/shared/path-util.c b/src/shared/path-util.c
index b623fc3dd5..0c1b6a0ab0 100644
--- a/src/shared/path-util.c
+++ b/src/shared/path-util.c
@@ -50,7 +50,8 @@ char *path_get_file_name(const char *p) {
assert(p);
- if ((r = strrchr(p, '/')))
+ r = strrchr(p, '/');
+ if (r)
return r + 1;
return (char*) p;
diff --git a/src/shared/time-dst.c b/src/shared/time-dst.c
index afc893cccc..3950a882bf 100644
--- a/src/shared/time-dst.c
+++ b/src/shared/time-dst.c
@@ -38,6 +38,7 @@
#include <sys/stat.h>
#include "time-dst.h"
+#include "util.h"
/*
* If tzh_version is '2' or greater, the above is followed by a second instance
@@ -84,15 +85,12 @@ static inline int64_t decode64(const void *ptr) {
int time_get_dst(time_t date, const char *tzfile,
time_t *switch_cur, char **zone_cur, bool *dst_cur,
time_t *switch_next, int *delta_next, char **zone_next, bool *dst_next) {
- time_t *transitions = NULL;
- size_t num_transitions = 0;
unsigned char *type_idxs = 0;
size_t num_types = 0;
struct ttinfo *types = NULL;
char *zone_names = NULL;
struct stat st;
size_t num_isstd, num_isgmt;
- FILE *f;
struct tzhead tzhead;
size_t chars;
size_t i;
@@ -102,22 +100,21 @@ int time_get_dst(time_t date, const char *tzfile,
size_t tzspec_len;
size_t num_leaps;
size_t lo, hi;
- int err = -EINVAL;
+ size_t num_transitions = 0;
+ _cleanup_free_ time_t *transitions = NULL;
+ _cleanup_fclose_ FILE *f;
f = fopen(tzfile, "re");
if (f == NULL)
return -errno;
- if (fstat(fileno(f), &st) < 0) {
- err = -errno;
- fclose(f);
- return err;
- }
+ if (fstat(fileno(f), &st) < 0)
+ return -errno;
read_again:
if (fread((void *)&tzhead, sizeof(tzhead), 1, f) != 1 ||
memcmp(tzhead.tzh_magic, TZ_MAGIC, sizeof(tzhead.tzh_magic)) != 0)
- goto lose;
+ return -EINVAL;
num_transitions = (size_t)decode(tzhead.tzh_timecnt);
num_types = (size_t)decode(tzhead.tzh_typecnt);
@@ -139,31 +136,31 @@ read_again:
+ chars
+ num_leaps * 8 + num_isstd + num_isgmt);
if (fseek(f, to_skip, SEEK_CUR) != 0)
- goto lose;
+ return -EINVAL;
goto read_again;
}
if (num_transitions > ((SIZE_MAX - (__alignof__(struct ttinfo) - 1)) / (sizeof(time_t) + 1)))
- goto lose;
+ return -EINVAL;
total_size = num_transitions * (sizeof(time_t) + 1);
total_size = ((total_size + __alignof__(struct ttinfo) - 1) & ~(__alignof__(struct ttinfo) - 1));
types_idx = total_size;
if (num_leaps > (SIZE_MAX - total_size) / sizeof(struct ttinfo))
- goto lose;
+ return -EINVAL;
total_size += num_types * sizeof(struct ttinfo);
if (chars > SIZE_MAX - total_size)
- goto lose;
+ return -EINVAL;
total_size += chars;
if (__alignof__(struct leap) - 1 > SIZE_MAX - total_size)
- goto lose;
+ return -EINVAL;
total_size = ((total_size + __alignof__(struct leap) - 1) & ~(__alignof__(struct leap) - 1));
if (num_leaps > (SIZE_MAX - total_size) / sizeof(struct leap))
- goto lose;
+ return -EINVAL;
total_size += num_leaps * sizeof(struct leap);
tzspec_len = 0;
@@ -171,24 +168,24 @@ read_again:
off_t rem = st.st_size - ftello(f);
if (rem < 0 || (size_t) rem < (num_transitions * (8 + 1) + num_types * 6 + chars))
- goto lose;
+ return -EINVAL;
tzspec_len = (size_t) rem - (num_transitions * (8 + 1) + num_types * 6 + chars);
if (num_leaps > SIZE_MAX / 12 || tzspec_len < num_leaps * 12)
- goto lose;
+ return -EINVAL;
tzspec_len -= num_leaps * 12;
if (tzspec_len < num_isstd)
- goto lose;
+ return -EINVAL;
tzspec_len -= num_isstd;
if (tzspec_len == 0 || tzspec_len - 1 < num_isgmt)
- goto lose;
+ return -EINVAL;
tzspec_len -= num_isgmt + 1;
if (SIZE_MAX - total_size < tzspec_len)
- goto lose;
+ return -EINVAL;
}
transitions = (time_t *)calloc(total_size + tzspec_len, 1);
if (transitions == NULL)
- goto lose;
+ return -EINVAL;
type_idxs = (unsigned char *)transitions + (num_transitions
* sizeof(time_t));
@@ -197,18 +194,18 @@ read_again:
if (sizeof(time_t) == 4 || trans_width == 8) {
if (fread(transitions, trans_width + 1, num_transitions, f) != num_transitions)
- goto lose;
+ return -EINVAL;
} else {
if (fread(transitions, 4, num_transitions, f) != num_transitions ||
fread(type_idxs, 1, num_transitions, f) != num_transitions)
- goto lose;
+ return -EINVAL;
}
/* Check for bogus indices in the data file, so we can hereafter
safely use type_idxs[T] as indices into `types' and never crash. */
for (i = 0; i < num_transitions; ++i)
if (type_idxs[i] >= num_types)
- goto lose;
+ return -EINVAL;
if ((BYTE_ORDER != BIG_ENDIAN && (sizeof(time_t) == 4 || trans_width == 4)) ||
(BYTE_ORDER == BIG_ENDIAN && sizeof(time_t) == 8 && trans_width == 4)) {
@@ -231,26 +228,26 @@ read_again:
int c;
if (fread(x, 1, sizeof(x), f) != sizeof(x))
- goto lose;
+ return -EINVAL;
c = getc(f);
if ((unsigned int)c > 1u)
- goto lose;
+ return -EINVAL;
types[i].isdst = c;
c = getc(f);
if ((size_t) c > chars)
/* Bogus index in data file. */
- goto lose;
+ return -EINVAL;
types[i].idx = c;
types[i].offset = (long int)decode(x);
}
if (fread(zone_names, 1, chars, f) != chars)
- goto lose;
+ return -EINVAL;
for (i = 0; i < num_isstd; ++i) {
int c = getc(f);
if (c == EOF)
- goto lose;
+ return -EINVAL;
types[i].isstd = c != 0;
}
@@ -260,7 +257,7 @@ read_again:
for (i = 0; i < num_isgmt; ++i) {
int c = getc(f);
if (c == EOF)
- goto lose;
+ return -EINVAL;
types[i].isgmt = c != 0;
}
@@ -268,10 +265,10 @@ read_again:
types[i++].isgmt = 0;
if (num_transitions == 0)
- goto lose;
+ return -EINVAL;
if (date < transitions[0] || date >= transitions[num_transitions - 1])
- goto lose;
+ return -EINVAL;
/* Find the first transition after TIMER, and
then pick the type of the transition before it. */
@@ -331,11 +328,5 @@ found:
if (dst_next)
*dst_next = types[type_idxs[i]].isdst;
- free(transitions);
- fclose(f);
return 0;
-lose:
- free(transitions);
- fclose(f);
- return err;
}