[PATCH 00/12] multipath-tools: Coverity patches

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Christophe,

here are a few simple fixes motivated by a recent
coverity scan of the latest upstream. Most are false positives,
but it can't hurt to fix them anyway, be it only to get a quicker
overview on future scans.

Unfortunately synopsys messed up the scan.coverity.com website
before I could finish this work.

There are two bigger issues with coverity that this set doesn't
address:

1) apparently coverity doesn't understand our frequently
    used paradigm (simplified):

    pthread_cleanup_push(pthread_mutex_unlock, &mutex);
    pthread_mutex_lock(&mutex);
    do_some_work();
    pthread_cleanup_pop(1);

 2) coverity warns that we sleep in multiple places holding the
    vecs lock. That will be a bit harder to fix.

Martin

Martin Wilck (12):
  kpartx(coverity): fix resource leak warning
  libmultipath(coverity): cleanup dup usage in execute_program()
  kpartx(coverity): fix apparent out-of-bounds access
  libmultipath(coverity): make sure readlink result is 0-terminated
  libmultipath(coverity): fix apparent overflow
  libmpathcmd(coverity): limit reply length
  libmultipath(coverity): fix int overflow in sysfs_set_scsi_tmo
  libmultipath(coverity): fix "enum misuse" for find_multipaths
  multipathd(coverity): check retval clock_gettime()
  libmpathpersist(coverity): range checking for PRIN length
  libmultipath/foreign(coverity): retval check in snprint_nvme_path
  libmultipath(coverity): fix possible NULL dereference

 kpartx/dasd.c                    |  7 +++----
 libmpathcmd/mpath_cmd.c          |  4 ++++
 libmpathcmd/mpath_cmd.h          |  6 ++++++
 libmpathpersist/mpath_pr_ioctl.c | 10 +++++++---
 libmultipath/callout.c           | 13 +++++++------
 libmultipath/configure.c         |  2 +-
 libmultipath/dict.c              |  2 +-
 libmultipath/discovery.c         |  4 ++--
 libmultipath/foreign/nvme.c      |  6 ++++--
 libmultipath/util.c              |  1 +
 multipathd/cli.c                 |  2 ++
 multipathd/cli.h                 |  6 ++++++
 multipathd/cli_handlers.c        |  1 +
 multipathd/main.c                |  9 +++++----
 14 files changed, 50 insertions(+), 23 deletions(-)

-- 
2.19.2

--
dm-devel mailing list
dm-devel@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/dm-devel



[Index of Archives]     [DM Crypt]     [Fedora Desktop]     [ATA RAID]     [Fedora Marketing]     [Fedora Packaging]     [Fedora SELinux]     [Yosemite Discussion]     [KDE Users]     [Fedora Docs]

  Powered by Linux