summaryrefslogtreecommitdiff
path: root/src/timezone.c
diff options
context:
space:
mode:
Diffstat (limited to 'src/timezone.c')
-rw-r--r--src/timezone.c38
1 files changed, 19 insertions, 19 deletions
diff --git a/src/timezone.c b/src/timezone.c
index 2b556c28..e346b11a 100644
--- a/src/timezone.c
+++ b/src/timezone.c
@@ -60,7 +60,7 @@ static char *read_key_file(const char *pathname, const char *key)
}
map = mmap(0, st.st_size, PROT_READ, MAP_SHARED, fd, 0);
- if (map == NULL || map == MAP_FAILED) {
+ if (!map || map == MAP_FAILED) {
close(fd);
return NULL;
}
@@ -81,25 +81,25 @@ static char *read_key_file(const char *pathname, const char *key)
}
ptr = memchr(ptr + 1, key[0], ptrlen - 1);
- if (ptr == NULL)
+ if (!ptr)
break;
ptrlen = st.st_size - (ptr - map);
}
- if (ptr != NULL) {
+ if (ptr) {
char *end, *val;
ptrlen = st.st_size - (ptr - map);
end = memchr(ptr, '\n', ptrlen);
- if (end != NULL)
+ if (end)
ptrlen = end - ptr;
val = memchr(ptr, '"', ptrlen);
- if (val != NULL) {
+ if (val) {
end = memchr(val + 1, '"', end - val - 1);
- if (end != NULL)
+ if (end)
str = g_strndup(val + 1, end - val - 1);
else
str = NULL;
@@ -137,10 +137,10 @@ static int compare_file(void *src_map, struct stat *src_st,
}
dst_map = mmap(0, dst_st.st_size, PROT_READ, MAP_SHARED, fd, 0);
- if (dst_map == NULL || dst_map == MAP_FAILED) {
+ if (!dst_map || dst_map == MAP_FAILED) {
close(fd);
return -1;
- }
+ }
result = memcmp(src_map, dst_map, src_st->st_size);
@@ -160,14 +160,14 @@ static char *find_origin(void *src_map, struct stat *src_st,
struct stat buf;
int ret;
- if (subpath == NULL)
- strncpy(pathname, basepath, sizeof(pathname));
+ if (!subpath)
+ strncpy(pathname, basepath, sizeof(pathname) - 1);
else
snprintf(pathname, sizeof(pathname),
"%s/%s", basepath, subpath);
dir = opendir(pathname);
- if (dir == NULL)
+ if (!dir)
return NULL;
while ((d = readdir(dir))) {
@@ -179,7 +179,7 @@ static char *find_origin(void *src_map, struct stat *src_st,
switch (d->d_type) {
case DT_REG:
- if (subpath == NULL)
+ if (!subpath)
snprintf(pathname, PATH_MAX,
"%s/%s", basepath, d->d_name);
else
@@ -206,14 +206,14 @@ static char *find_origin(void *src_map, struct stat *src_st,
continue;
/* fall through */
case DT_DIR:
- if (subpath == NULL)
+ if (!subpath)
strncpy(pathname, d->d_name, sizeof(pathname));
else
snprintf(pathname, sizeof(pathname),
"%s/%s", subpath, d->d_name);
str = find_origin(src_map, src_st, basepath, pathname);
- if (str != NULL) {
+ if (str) {
closedir(dir);
return str;
}
@@ -248,14 +248,14 @@ char *__connman_timezone_lookup(void)
if (S_ISREG(st.st_mode)) {
map = mmap(0, st.st_size, PROT_READ, MAP_SHARED, fd, 0);
- if (map == NULL || map == MAP_FAILED) {
+ if (!map || map == MAP_FAILED) {
g_free(zone);
zone = NULL;
goto done;
}
- if (zone != NULL) {
+ if (zone) {
char pathname[PATH_MAX];
snprintf(pathname, PATH_MAX, "%s/%s",
@@ -267,7 +267,7 @@ char *__connman_timezone_lookup(void)
}
}
- if (zone == NULL)
+ if (!zone)
zone = find_origin(map, &st, USR_SHARE_ZONEINFO, NULL);
munmap(map, st.st_size);
@@ -331,7 +331,7 @@ int __connman_timezone_change(const char *zone)
}
map = mmap(0, st.st_size, PROT_READ, MAP_SHARED, fd, 0);
- if (map == NULL || map == MAP_FAILED) {
+ if (!map || map == MAP_FAILED) {
close(fd);
return -EIO;
}
@@ -415,7 +415,7 @@ int __connman_timezone_init(void)
return -EIO;
channel = g_io_channel_unix_new(fd);
- if (channel == NULL) {
+ if (!channel) {
close(fd);
return -EIO;
}