On Tue, Sep 15, 2020 at 12:38:27PM +0800, lixiaokeng wrote: > Hi, > The udev* function may return NULL,and it will be > dereferenced in str* and sscanf func. For example, > there is a coredump caused in add func, which show in > be7a043(commit id) in upstream-queue. We check the > return value to avoid dereference NULL. > > repo: openSUSE/multipath-tools > repo link: https://github.com/openSUSE/multipath-tools > branch: upstream-queue > For the set Reviewed-by: Benjamin Marzinski <bmarzins@xxxxxxxxxx> > lixiaokeng (6): > libmultipath: check uedv* return value in sysfs_get_host_pci_name > libmultipath: check udev* return value in ccw_sysfs_pathinfo > libmultipath: check udev* return value in sysfs_get_tgt_nodename > libmultipath: check udev* return value in > trigger_partitions_udev_change > libmultipath: check udev* renturn value in get_ctrl_blkdev > libmultipath: check udev* return value in _find_path_by_syspath > > libmultipath/configure.c | 4 +++- > libmultipath/discovery.c | 9 +++++++-- > libmultipath/foreign/nvme.c | 10 +++++++--- > 3 files changed, 17 insertions(+), 6 deletions(-) -- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel
