mkfs.f2fs: fix wrong curseg check

We should avoid i==j, otherwise we always assign 0~5 segments.

Reviewed-by: Chao Yu <yuchao0@huawei.com>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>

diff --git a/mkfs/f2fs_format.c b/mkfs/f2fs_format.c
index b379e80..b57f925 100644 (file)
--- a/mkfs/f2fs_format.c
+++ b/mkfs/f2fs_format.c
@@ -120,14 +120,18 @@ next:
 static void verify_cur_segs(void)
 {
        int i, j;
+       int reorder = 0;
 
        for (i = 0; i < NR_CURSEG_TYPE; i++) {
-               for (j = 0; j < NR_CURSEG_TYPE; j++)
-                       if (c.cur_seg[i] == c.cur_seg[j])
+               for (j = i + 1; j < NR_CURSEG_TYPE; j++) {
+                       if (c.cur_seg[i] == c.cur_seg[j]) {
+                               reorder = 1;
                                break;
+                       }
+               }
        }
 
-       if (i == NR_CURSEG_TYPE && j == NR_CURSEG_TYPE)
+       if (!reorder)
                return;
 
        c.cur_seg[0] = 0;