diff options
author | Minghao Chi <chi.minghao@zte.com.cn> | 2022-04-29 07:45:56 +0200 |
---|---|---|
committer | Richard Weinberger <richard@nod.at> | 2022-05-27 16:21:20 +0200 |
commit | 5bff56edab7f11bda31cfc708634a025bb265836 (patch) | |
tree | f79adeabff61a9c6b9c7a6b70d4cd39babceb010 /fs/ubifs/budget.c | |
parent | jffs2: fix memory leak in jffs2_do_fill_super (diff) | |
download | linux-5bff56edab7f11bda31cfc708634a025bb265836.tar.xz linux-5bff56edab7f11bda31cfc708634a025bb265836.zip |
ubifs: Simplify the return expression of run_gc()
Simplify the return expression.
Reported-by: Zeal Robot <zealci@zte.com.cn>
Signed-off-by: Minghao Chi <chi.minghao@zte.com.cn>
Reviewed-by: Tudor Ambarus <tudor.ambarus@microchip.com>
Signed-off-by: Richard Weinberger <richard@nod.at>
Diffstat (limited to 'fs/ubifs/budget.c')
-rw-r--r-- | fs/ubifs/budget.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/fs/ubifs/budget.c b/fs/ubifs/budget.c index c0b84e960b20..e8b9b756f0ac 100644 --- a/fs/ubifs/budget.c +++ b/fs/ubifs/budget.c @@ -65,7 +65,7 @@ static void shrink_liability(struct ubifs_info *c, int nr_to_write) */ static int run_gc(struct ubifs_info *c) { - int err, lnum; + int lnum; /* Make some free space by garbage-collecting dirty space */ down_read(&c->commit_sem); @@ -76,10 +76,7 @@ static int run_gc(struct ubifs_info *c) /* GC freed one LEB, return it to lprops */ dbg_budg("GC freed LEB %d", lnum); - err = ubifs_return_leb(c, lnum); - if (err) - return err; - return 0; + return ubifs_return_leb(c, lnum); } /** |