[checkers]
authorBenjamin Marzinski <bmarzins@redhat.com>
Tue, 13 May 2008 22:36:29 +0000 (00:36 +0200)
committerChristophe Varoqui <christophe.varoqui@free.fr>
Tue, 13 May 2008 22:36:29 +0000 (00:36 +0200)
commit81f2f0bf9a015bad2a130f0d1281fdf68a6b1a17
tree8d146d649e75fb5912036343f3705f398971937c
parentcf94ef0ee3f8046aa2b8dc90aee14edf76d4bbf7
[checkers]

checker_lookup() silently returns the default checker if the requested
one can't be loaded. Even worse, it the default checker can't be loaded,
it gets stuck in an infinite loop.

It seems to me that if the requested checker can't be loaded, we should
just throw an error. That's what this patch does.

multipath.conf should be order independent, and with the existing
checker code, it isn't.  The multipath_dir had to be set before
you could set a checker, otherwise the checker load failed (since the
multipath_dir wasn't specified).
libmultipath/checkers.c
libmultipath/checkers.h
libmultipath/config.c
libmultipath/config.h
libmultipath/dict.c
libmultipath/hwtable.c
libmultipath/propsel.c