fix jffs2reset on ubifs overlay

change find_mount_point parameter to match either all or only
jffs2 and ubifs, which are typically used on embedded mtd devices.
For now, find_mount_point was used without the fstype parameter
except for a single occurance in jffs2reset. There it is desireable
to apply the same logic as previously hard-coded to jffs2 also in the
case of a ubifs overlay, so firstboot aka. factory-reset works as
expected.

Signed-off-by: Daniel Golle <daniel@makrotopia.org>

diff --git a/jffs2reset.c b/jffs2reset.c
index 122e03b123afc3b431ec4bfc7554279419030302..56848723a0a005530188d5cc9c26d329c651dc70 100644 (file)
--- a/jffs2reset.c
+++ b/jffs2reset.c
@@ -84,7 +84,7 @@ jffs2_reset(int argc, char **argv)
                return -1;
        }
 
-       mp = find_mount_point(v->blk, "jffs2");
+       mp = find_mount_point(v->blk, 1);
        if (mp) {
                fprintf(stderr, "%s is mounted as %s, only erasing files\n", v->blk, mp);
                foreachdir(mp, handle_rmdir);

diff --git a/libfstools/find.c b/libfstools/find.c
index 71548f5bfcb5a942135bed92f412ced50f83c057..d9df173025673a6c04484491d2659e0c3b817bb8 100644 (file)
--- a/libfstools/find.c
+++ b/libfstools/find.c
@@ -73,7 +73,7 @@ find_mount(char *mp)
 }
 
 char*
-find_mount_point(char *block, char *fs)
+find_mount_point(char *block, int mtd_only)
 {
        FILE *fp = fopen("/proc/mounts", "r");
        static char line[256];
@@ -96,7 +96,9 @@ find_mount_point(char *block, char *fs)
                        *t = '\0';
                        t++;
 
-                       if (fs && strncmp(t, fs, strlen(fs))) {
+                       if (mtd_only &&
+                           strncmp(t, "jffs2", 6) &&
+                           strncmp(t, "ubifs", 6)) {
                                fclose(fp);
                                fprintf(stderr, "block is mounted with wrong fs\n");
                                return NULL;

diff --git a/libfstools/jffs2.c b/libfstools/jffs2.c
index 0da9f2f75e463ab8de98dcf9b42443c0f09759d0..134e09dd269ec8b07ead380bae773b8d86fe6486 100644 (file)
--- a/libfstools/jffs2.c
+++ b/libfstools/jffs2.c
@@ -172,7 +172,7 @@ jffs2_switch(int argc, char **argv)
        }
 
        v = volume_find("rootfs_data");
-       mp = find_mount_point(v->blk, NULL);
+       mp = find_mount_point(v->blk, 0);
        if (mp) {
                fprintf(stderr, "rootfs_data:%s is already mounted as %s\n", v->blk, mp);
                return -1;
@@ -240,7 +240,7 @@ int mount_overlay(void)
        if (!v)
                return -1;
 
-       mp = find_mount_point(v->blk, NULL);
+       mp = find_mount_point(v->blk, 0);
        if (mp) {
                fprintf(stderr, "rootfs_data:%s is already mounted as %s\n", v->blk, mp);
                return -1;

diff --git a/libfstools/libfstools.h b/libfstools/libfstools.h
index f9cafbde126143d63920a7fbf73417fc62560f53..70220c25a7efee5c86882683eff9fce2948d536c 100644 (file)
--- a/libfstools/libfstools.h
+++ b/libfstools/libfstools.h
@@ -37,7 +37,7 @@ extern int ramoverlay(void);
 
 extern int find_overlay_mount(char *overlay);
 extern char* find_mount(char *mp);
-extern char* find_mount_point(char *block, char *fs);
+extern char* find_mount_point(char *block, int mtd_only);
 extern int find_filesystem(char *fs);
 extern int find_mtd_block(char *name, char *part, int plen);
 extern int find_mtd_char(char *name, char *part, int plen);

diff --git a/libfstools/overlay.c b/libfstools/overlay.c
index ae0ead3b4714d81b3657262855df1a66c6b9e503..dc350b5ca5fb5998ee27684d514a1dfa3474d9d1 100644 (file)
--- a/libfstools/overlay.c
+++ b/libfstools/overlay.c
@@ -158,7 +158,7 @@ jffs2_switch(int argc, char **argv)
        }
 
        v = volume_find("rootfs_data");
-       mp = find_mount_point(v->blk, NULL);
+       mp = find_mount_point(v->blk, 0);
        if (mp) {
                fprintf(stderr, "rootfs_data:%s is already mounted as %s\n", v->blk, mp);
                return -1;
@@ -244,7 +244,7 @@ int mount_overlay(void)
        if (!v)
                return -1;
 
-       mp = find_mount_point(v->blk, NULL);
+       mp = find_mount_point(v->blk, 0);
        if (mp) {
                fprintf(stderr, "rootfs_data:%s is already mounted as %s\n", v->blk, mp);
                return -1;