diff options
| author | Pawel Jakub Dawidek <pjd@FreeBSD.org> | 2010-08-31 19:01:46 +0000 |
|---|---|---|
| committer | Pawel Jakub Dawidek <pjd@FreeBSD.org> | 2010-08-31 19:01:46 +0000 |
| commit | 81bbc7f7494264682705579f7313ec4a24c7b586 (patch) | |
| tree | 4b004d806942637cf89f5207155bdb78d9dd4c4d /cddl | |
| parent | cff47502332de59b864b682851f99d954d8fcc45 (diff) | |
| download | src-81bbc7f7494264682705579f7313ec4a24c7b586.tar.gz src-81bbc7f7494264682705579f7313ec4a24c7b586.zip | |
Eliminate confusing while () loop. In the first version of the code it was
there to avoid gotos, but in the current version it serves no purpose.
MFC after: 2 weeks
Notes
Notes:
svn path=/head/; revision=212067
Diffstat (limited to 'cddl')
| -rw-r--r-- | cddl/contrib/opensolaris/cmd/zpool/zpool_main.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/cddl/contrib/opensolaris/cmd/zpool/zpool_main.c b/cddl/contrib/opensolaris/cmd/zpool/zpool_main.c index 2d9891674a7b..08d8afeb0b4b 100644 --- a/cddl/contrib/opensolaris/cmd/zpool/zpool_main.c +++ b/cddl/contrib/opensolaris/cmd/zpool/zpool_main.c @@ -3347,20 +3347,19 @@ is_root_pool(zpool_handle_t *zhp) static boolean_t stated = B_FALSE; char *slash; - while (!stated) { + if (!stated) { stated = B_TRUE; if (statfs("/", &sfs) == -1) { (void) fprintf(stderr, "Unable to stat root file system: %s.\n", strerror(errno)); - break; + return (0); } if (strcmp(sfs.f_fstypename, "zfs") != 0) - break; + return (0); poolname = sfs.f_mntfromname; if ((slash = strchr(poolname, '/')) != NULL) *slash = '\0'; - break; } return (poolname != NULL && strcmp(poolname, zpool_get_name(zhp)) == 0); } |