summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDenys Vlasenko2010-03-05 13:20:28 +0100
committerDenys Vlasenko2010-03-05 13:20:28 +0100
commit44fbfa78ca6126d658ff48cef79e2f7d29fae6c8 (patch)
tree4eff59dd3aff3fe6a46df9bf6cc104a3d3c1ec61
parent917693b5b3d1ece0ade489bc4d922bfa368b979e (diff)
downloadbusybox-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.c31
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 */