Since md is freed before the do-while checks it, it's better to change it to while(1). Signed-off-by: Shen Feng <shen@xxxxxxxxxxxxxx> --- fs/ext4/mballoc.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/fs/ext4/mballoc.c b/fs/ext4/mballoc.c index 42553f6..063f820 100644 --- a/fs/ext4/mballoc.c +++ b/fs/ext4/mballoc.c @@ -2572,7 +2572,7 @@ ext4_mb_free_committed_blocks(struct super_block *sb) kfree(md); ext4_mb_release_desc(&e4b); - } while (md); + } while (1); mb_debug("freed %u blocks in %u structures\n", count, count2); } -- 1.5.4.5 -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html