Re: [PATCH] libmultipath: remove unnecessary condition in checker_repair

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Merged,
thanks.

On Mon, Nov 7, 2016 at 9:57 AM, <peng.liang5@xxxxxxxxxx> wrote:
From: PengLiang <peng.liang5@xxxxxxxxxx>

The checker's validity is an unnecessary checking. It will be checked in checker_selected.

Signed-off-by: PengLiang <peng.liang5@xxxxxxxxxx>
---
 libmultipath/checkers.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libmultipath/checkers.c b/libmultipath/checkers.c
index fd999b0..05e024f 100644
--- a/libmultipath/checkers.c
+++ b/libmultipath/checkers.c
@@ -213,7 +213,7 @@ void checker_put (struct checker * dst)

 void checker_repair (struct checker * c)
 {
-       if (!c || !checker_selected(c))
+       if (!checker_selected(c))
                return;

        c->message[0] = '\0';
--
2.8.1.windows.1



--
dm-devel mailing list
dm-devel@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/dm-devel

[Index of Archives]     [DM Crypt]     [Fedora Desktop]     [ATA RAID]     [Fedora Marketing]     [Fedora Packaging]     [Fedora SELinux]     [Yosemite Discussion]     [KDE Users]     [Fedora Docs]

  Powered by Linux