mpp->wwid is a character array in the multipath struction, not a pointer, so it is never NULL. multipath needs to check if the string is empty instead. Signed-off-by: Benjamin Marzinski <bmarzins@xxxxxxxxxx> --- libmultipath/structs_vec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libmultipath/structs_vec.c b/libmultipath/structs_vec.c index 7073915..72622b0 100644 --- a/libmultipath/structs_vec.c +++ b/libmultipath/structs_vec.c @@ -106,7 +106,7 @@ orphan_paths (vector pathvec, struct multipath * mpp) static void set_multipath_wwid (struct multipath * mpp) { - if (mpp->wwid) + if (strlen(mpp->wwid)) return; dm_get_uuid(mpp->alias, mpp->wwid); -- 1.8.2 -- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel