Commit 009b8109 authored by ZhangXiaoxu's avatar ZhangXiaoxu Committed by Khalid Elmously

dm space map metadata: fix missing store of apply_bops() return value

BugLink: https://bugs.launchpad.net/bugs/1845036

commit ae148243 upstream.

In commit 6096d91a ("dm space map metadata: fix occasional leak
of a metadata block on resize"), we refactor the commit logic to a new
function 'apply_bops'.  But when that logic was replaced in out() the
return value was not stored.  This may lead out() returning a wrong
value to the caller.

Fixes: 6096d91a ("dm space map metadata: fix occasional leak of a metadata block on resize")
Cc: stable@vger.kernel.org
Signed-off-by: default avatarZhangXiaoxu <zhangxiaoxu5@huawei.com>
Signed-off-by: default avatarMike Snitzer <snitzer@redhat.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: default avatarConnor Kuehl <connor.kuehl@canonical.com>
Signed-off-by: default avatarKhalid Elmously <khalid.elmously@canonical.com>
parent 0082cc87
...@@ -248,7 +248,7 @@ static int out(struct sm_metadata *smm) ...@@ -248,7 +248,7 @@ static int out(struct sm_metadata *smm)
} }
if (smm->recursion_count == 1) if (smm->recursion_count == 1)
apply_bops(smm); r = apply_bops(smm);
smm->recursion_count--; smm->recursion_count--;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment