diff options
| author | Denys Vlasenko | 2010-03-05 13:20:28 +0100 |
|---|---|---|
| committer | Denys Vlasenko | 2010-03-05 13:20:28 +0100 |
| commit | 44fbfa78ca6126d658ff48cef79e2f7d29fae6c8 (patch) | |
| tree | 4eff59dd3aff3fe6a46df9bf6cc104a3d3c1ec61 | |
| parent | 917693b5b3d1ece0ade489bc4d922bfa368b979e (diff) | |
| download | busybox-44fbfa78ca6126d658ff48cef79e2f7d29fae6c8.zip busybox-44fbfa78ca6126d658ff48cef79e2f7d29fae6c8.tar.gz | |
libbb/loop.c: style and readability fixes, no code changes
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
| -rw-r--r-- | libbb/loop.c | 31 |
1 files changed, 18 insertions, 13 deletions
diff --git a/libbb/loop.c b/libbb/loop.c index 3fec7ad..a11933f 100644 --- a/libbb/loop.c +++ b/libbb/loop.c @@ -56,14 +56,16 @@ char* FAST_FUNC query_loop(const char *device) { int fd; bb_loop_info loopinfo; - char *dev = 0; + char *dev = NULL; fd = open(device, O_RDONLY); - if (fd < 0) return 0; - if (!ioctl(fd, BB_LOOP_GET_STATUS, &loopinfo)) - dev = xasprintf("%ld %s", (long) loopinfo.lo_offset, - (char *)loopinfo.lo_file_name); - close(fd); + if (fd >= 0) { + if (ioctl(fd, BB_LOOP_GET_STATUS, &loopinfo) == 0) { + dev = xasprintf("%lu %s", (long) loopinfo.lo_offset, + (char *)loopinfo.lo_file_name); + } + close(fd); + } return dev; } @@ -73,7 +75,8 @@ int FAST_FUNC del_loop(const char *device) int fd, rc; fd = open(device, O_RDONLY); - if (fd < 0) return 1; + if (fd < 0) + return 1; rc = ioctl(fd, LOOP_CLR_FD, 0); close(fd); @@ -110,7 +113,7 @@ int FAST_FUNC set_loop(char **device, const char *file, unsigned long long offse sprintf(dev, LOOP_FORMAT, i); /* Ran out of block devices, return failure. */ - if (stat(try, &statbuf) || !S_ISBLK(statbuf.st_mode)) { + if (stat(try, &statbuf) != 0 || !S_ISBLK(statbuf.st_mode)) { rc = -ENOENT; break; } @@ -131,8 +134,8 @@ int FAST_FUNC set_loop(char **device, const char *file, unsigned long long offse safe_strncpy((char *)loopinfo.lo_file_name, file, LO_NAME_SIZE); loopinfo.lo_offset = offset; /* Associate free loop device with file. */ - if (!ioctl(dfd, LOOP_SET_FD, ffd)) { - if (!ioctl(dfd, BB_LOOP_SET_STATUS, &loopinfo)) + if (ioctl(dfd, LOOP_SET_FD, ffd) == 0) { + if (ioctl(dfd, BB_LOOP_SET_STATUS, &loopinfo) == 0) rc = 0; else ioctl(dfd, LOOP_CLR_FD, 0); @@ -143,8 +146,10 @@ int FAST_FUNC set_loop(char **device, const char *file, unsigned long long offse file isn't pretty either. In general, mounting the same file twice without using losetup manually is problematic.) */ - } else if (strcmp(file, (char *)loopinfo.lo_file_name) != 0 - || offset != loopinfo.lo_offset) { + } else + if (strcmp(file, (char *)loopinfo.lo_file_name) != 0 + || offset != loopinfo.lo_offset + ) { rc = -1; } close(dfd); @@ -152,7 +157,7 @@ int FAST_FUNC set_loop(char **device, const char *file, unsigned long long offse if (*device) break; } close(ffd); - if (!rc) { + if (rc == 0) { if (!*device) *device = xstrdup(dev); return (mode == O_RDONLY); /* 1:ro, 0:rw */ |