summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorПреподобный Гомер <mk.43.ecko@gmail.com>2016-08-17 14:51:31 +0300
committerJohn Crispin <john@phrozen.org>2016-08-18 15:25:26 +0200
commit3f0e8e1a08a7dbfd408affeece24f626bf4a54b8 (patch)
tree33604d3045fc58f2cd4c705e691e3260e2a62f01
parent532a5568104449777d24c4f83f24b9c9581d54ab (diff)
downloadfstools-3f0e8e1a08a7dbfd408affeece24f626bf4a54b8.tar.gz
libfstools: "lowerdir" variable name in fopivot
Hi all! In fopivot in libfstools variable name "lowerdir" is confusing and doesn't make any sense. There is a patch renaming that var to "mount_options".
-rw-r--r--libfstools/mount.c14
1 files changed, 7 insertions, 7 deletions
diff --git a/libfstools/mount.c b/libfstools/mount.c
index 81176ce..371082b 100644
--- a/libfstools/mount.c
+++ b/libfstools/mount.c
@@ -82,7 +82,7 @@ pivot(char *new, char *old)
int
fopivot(char *rw_root, char *ro_root)
{
- char overlay[64], lowerdir[64];
+ char overlay[64], mount_options[64];
if (find_filesystem("overlay")) {
ULOG_ERR("BUG: no suitable fs found\n");
@@ -96,8 +96,8 @@ fopivot(char *rw_root, char *ro_root)
* If it fails, it means that we are probably using a v23 and
* later versions that require a workdir
*/
- snprintf(lowerdir, sizeof(lowerdir), "lowerdir=/,upperdir=%s", rw_root);
- if (mount(overlay, "/mnt", "overlayfs", MS_NOATIME, lowerdir)) {
+ snprintf(mount_options, sizeof(mount_options), "lowerdir=/,upperdir=%s", rw_root);
+ if (mount(overlay, "/mnt", "overlayfs", MS_NOATIME, mount_options)) {
char upperdir[64], workdir[64], upgrade[64], upgrade_dest[64];
struct stat st;
@@ -105,7 +105,7 @@ fopivot(char *rw_root, char *ro_root)
snprintf(workdir, sizeof(workdir), "%s/work", rw_root);
snprintf(upgrade, sizeof(upgrade), "%s/sysupgrade.tgz", rw_root);
snprintf(upgrade_dest, sizeof(upgrade_dest), "%s/sysupgrade.tgz", upperdir);
- snprintf(lowerdir, sizeof(lowerdir), "lowerdir=/,upperdir=%s,workdir=%s",
+ snprintf(mount_options, sizeof(mount_options), "lowerdir=/,upperdir=%s,workdir=%s",
upperdir, workdir);
/*
@@ -122,10 +122,10 @@ fopivot(char *rw_root, char *ro_root)
rename(upgrade, upgrade_dest);
/* Mainlined overlayfs has been renamed to "overlay", try that first */
- if (mount(overlay, "/mnt", "overlay", MS_NOATIME, lowerdir)) {
- if (mount(overlay, "/mnt", "overlayfs", MS_NOATIME, lowerdir)) {
+ if (mount(overlay, "/mnt", "overlay", MS_NOATIME, mount_options)) {
+ if (mount(overlay, "/mnt", "overlayfs", MS_NOATIME, mount_options)) {
ULOG_ERR("mount failed: %s, options %s\n",
- strerror(errno), lowerdir);
+ strerror(errno), mount_options);
return -1;
}
}