Previously detect_alua was setting pp->tpgs if the path state was either PATH_UP or PATH_REMOVED. Setting pp->tpgs for PATH_REMOVED makes no sense. PATH_UP is the only state that path_offline() returns where we should set pp->tpgs, so just check for that, instead of checking for all the states where we shouldn't set pp->tpgs. Signed-off-by: Benjamin Marzinski <bmarzins@xxxxxxxxxx> --- libmultipath/discovery.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libmultipath/discovery.c b/libmultipath/discovery.c index e2052422..380e0e95 100644 --- a/libmultipath/discovery.c +++ b/libmultipath/discovery.c @@ -1087,7 +1087,7 @@ detect_alua(struct path * pp) return; state = path_offline(pp); - if (state == PATH_DOWN || state == PATH_PENDING) + if (state != PATH_UP) return; pp->tpgs = TPGS_NONE; -- 2.45.0
