From: Miaohe Lin Date: Tue, 15 Dec 2020 03:06:04 +0000 (-0800) Subject: mm/swapfile.c: remove unnecessary out label in __swap_duplicate() X-Git-Tag: Ubuntu-5.11.0-11.12~894^2~150 X-Git-Url: https://git.proxmox.com/?a=commitdiff_plain;h=9d9a03340309cb8065503cfa3c5c5fc8b7670230;p=mirror_ubuntu-hirsute-kernel.git mm/swapfile.c: remove unnecessary out label in __swap_duplicate() When the code went to the out label, it must have p == NULL. So what out label really does is redundant if check and return err. We should Remove this unnecessary out label because it does not handle resource free and so on. Link: https://lkml.kernel.org/r/20201009130337.29698-1-linmiaohe@huawei.com Signed-off-by: Miaohe Lin Reviewed-by: Andrew Morton Signed-off-by: Andrew Morton Signed-off-by: Linus Torvalds --- diff --git a/mm/swapfile.c b/mm/swapfile.c index 4a6ff685a736..83df467e1135 100644 --- a/mm/swapfile.c +++ b/mm/swapfile.c @@ -3445,11 +3445,11 @@ static int __swap_duplicate(swp_entry_t entry, unsigned char usage) unsigned long offset; unsigned char count; unsigned char has_cache; - int err = -EINVAL; + int err; p = get_swap_device(entry); if (!p) - goto out; + return -EINVAL; offset = swp_offset(entry); ci = lock_cluster_or_swap_info(p, offset); @@ -3496,7 +3496,6 @@ static int __swap_duplicate(swp_entry_t entry, unsigned char usage) unlock_out: unlock_cluster_or_swap_info(p, ci); -out: if (p) put_swap_device(p); return err;