This is collection of issues found by coverity. The first three patches
deal with ev_remove_path() removing the path, but returning failure,
causing a use-after-free error. The next two patches fix memory leaks.
The final patch removes an unnecessary call to rescan_paths() from
uev_update_path()
Changes from v3:
0003: Handle NULL returns from strdup(). Suggested by Martin Wilck
Changes from v2:
0003: Combined with v2 patch 0006, with changes changes in comments,
how the return codes are defined, and how they are handled, based
on Martin's suggestions
0006: New patch based on Martin's suggestions about wwid change handling
in uev_update_path().
Changes from v1:
0001: changed comment based on Martin's suggestion
0004: moved location of atexit() call based on Martin's suggestion
0006: New patch, based on Martin's comments on patch 0003
Benjamin Marzinski (6):
multipathd: don't fail to remove path once the map is removed
multipathd: remove duplicate orphan_paths in flush_map
multipathd: fix ev_remove_path return code handling
multipath: free vectors in configure
kpartx: Don't leak memory when getblock returns NULL
multipathd: don't rescan_path on wwid change in uev_update_path
kpartx/kpartx.c | 2 ++
libmultipath/structs_vec.c | 4 +--
multipath/main.c | 7 ++++-
multipathd/cli_handlers.c | 24 ++++++++++++++--
multipathd/main.c | 56 ++++++++++++++++++++------------------
multipathd/main.h | 14 ++++++++++
6 files changed, 75 insertions(+), 32 deletions(-)
--
2.17.2
--
dm-devel mailing list
dm-devel@xxxxxxxxxx
https://listman.redhat.com/mailman/listinfo/dm-devel
