From mboxrd@z Thu Jan 1 00:00:00 1970 From: lixiaokeng Subject: [PATCH 0/6] libmultipath: check udev* func return value Date: Tue, 15 Sep 2020 12:38:27 +0800 Message-ID: Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dm-devel-bounces@redhat.com Errors-To: dm-devel-bounces@redhat.com Content-Language: en-GB To: Christophe Varoqui , Martin Wilck , Benjamin Marzinski , dm-devel mailing list Cc: linfeilong , "liuzhiqiang (I)" List-Id: dm-devel.ids 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 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(-)