diff options
| author | Sandeep Dhavale <[email protected]> | 2024-09-04 23:00:25 -0700 |
|---|---|---|
| committer | Gao Xiang <[email protected]> | 2024-09-10 00:46:34 +0800 |
| commit | 3fc3e45fcdeaad4b7660b560fcbc827eb733f58e (patch) | |
| tree | 88e9a4bc5eb27c5db300689afe428ce8ec058d63 | |
| parent | 59aadaa7ebafbc57e642d772cfc02c2b907e5b89 (diff) | |
erofs: fix error handling in z_erofs_init_decompressor
If we get a failure at the first decompressor init (i = 0),
the clean up while loop could enter infinite loop due to wrong while
check. Check the value of i now to see if we need any clean up at all.
Fixes: 5a7cce827ee9 ("erofs: refine z_erofs_{init,exit}_subsystem()")
Reported-by: liujinbao1 <[email protected]>
Signed-off-by: Sandeep Dhavale <[email protected]>
Reviewed-by: Gao Xiang <[email protected]>
Reviewed-by: Chao Yu <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Gao Xiang <[email protected]>
| -rw-r--r-- | fs/erofs/decompressor.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/erofs/decompressor.c b/fs/erofs/decompressor.c index c2253b6a5416..eb318c7ddd80 100644 --- a/fs/erofs/decompressor.c +++ b/fs/erofs/decompressor.c @@ -539,7 +539,7 @@ int __init z_erofs_init_decompressor(void) for (i = 0; i < Z_EROFS_COMPRESSION_MAX; ++i) { err = z_erofs_decomp[i] ? z_erofs_decomp[i]->init() : 0; if (err) { - while (--i) + while (i--) if (z_erofs_decomp[i]) z_erofs_decomp[i]->exit(); return err; |