From: Zdenek Kabelac <zkabelac@redhat.com>
To: lvm-devel@redhat.com
Subject: [PATCH 00/23] Static analyzis set
Date: Tue, 21 Dec 2010 16:41:32 +0100 [thread overview]
Message-ID: <cover.1292945707.git.zkabelac@redhat.com> (raw)
This is mostly the last bigger patch sets for covering
issues reported by Coverity tool.
Few patches needs some closer looking,
Most of them are trivial, but as they may present
some forgotten tests or missing code I'm posting
them for review.
Zdenek Kabelac (23):
Remove dead assignment to thisfd
Remove dead assignment
Remove dead assigment of mirr_state
Remove dead assignment of lock_flags
Remove dead assignment in vgconvert_single
Log of pthread_join operation
Move lastfd handling into for()
Fix memory leak in error path of restart_clvmd
Add test for errors in _setup_task
Fix memory leak of dev_filter on error path
Add stack traces for archive
Add tests for dm_task_set operations
Make it obvious this code is not used
Add stack trace for backup and backup_remove
Add test check for find_config_str
Remove check for vg != NULL
Add check for dm_snprintf result
Return PERCENT_INVALID for error case
Warning - dead code problem elimination
exit if lvmchache_init fails
Add default error path for get_property
Remove dead assignment in dm_event_get_registered_device
Remove dead assignment in vg_split_mdas
daemons/clvmd/clvmd-command.c | 3 +-
daemons/clvmd/clvmd.c | 25 ++++++++++---------
daemons/dmeventd/dmeventd.c | 3 +-
daemons/dmeventd/libdevmapper-event.c | 41 +++++++++++++++++---------------
lib/activate/dev_manager.c | 21 ++++++++++------
lib/commands/toolcontext.c | 2 +
lib/filters/filter-persistent.c | 22 ++++++++++++-----
lib/format_text/archiver.c | 9 +++++--
lib/format_text/import_vsn1.c | 18 ++++++--------
lib/locking/cluster_locking.c | 14 ++++++-----
lib/locking/file_locking.c | 16 +++++++++---
lib/metadata/metadata.c | 8 +++---
lib/mirror/mirrored.c | 3 --
lib/report/properties.c | 15 +++++++-----
liblvm/lvm_misc.c | 4 +++
tools/lvscan.c | 6 -----
tools/polldaemon.c | 3 +-
tools/vgconvert.c | 4 +--
tools/vgrename.c | 6 +++-
19 files changed, 125 insertions(+), 98 deletions(-)
--
1.7.3.4
next reply other threads:[~2010-12-21 15:41 UTC|newest]
Thread overview: 62+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-12-21 15:41 Zdenek Kabelac [this message]
2010-12-21 15:41 ` [PATCH 01/23] Remove dead assignment to thisfd Zdenek Kabelac
2011-01-05 2:40 ` Alasdair G Kergon
2010-12-21 15:41 ` [PATCH 02/23] Remove dead assignment Zdenek Kabelac
2011-01-05 2:44 ` Alasdair G Kergon
2010-12-21 15:41 ` [PATCH 03/23] Remove dead assigment of mirr_state Zdenek Kabelac
2011-01-05 2:38 ` Alasdair G Kergon
2010-12-21 15:41 ` [PATCH 04/23] Remove dead assignment of lock_flags Zdenek Kabelac
2010-12-21 16:20 ` Milan Broz
2010-12-22 10:11 ` Zdenek Kabelac
2011-01-05 0:30 ` Alasdair G Kergon
2011-01-05 0:57 ` Milan Broz
2010-12-21 15:41 ` [PATCH 05/23] Remove dead assignment in vgconvert_single Zdenek Kabelac
2011-01-05 1:11 ` Alasdair G Kergon
2010-12-21 15:41 ` [PATCH 06/23] Log of pthread_join operation Zdenek Kabelac
2011-01-05 2:36 ` Alasdair G Kergon
2010-12-21 15:41 ` [PATCH 07/23] Move lastfd handling into for() Zdenek Kabelac
2010-12-21 16:50 ` Milan Broz
2010-12-22 10:16 ` Zdenek Kabelac
2011-01-05 0:28 ` Alasdair G Kergon
2011-01-05 1:06 ` Alasdair G Kergon
2010-12-21 15:41 ` [PATCH 08/23] Fix memory leak in error path of restart_clvmd Zdenek Kabelac
2010-12-21 15:41 ` [PATCH 09/23] Add test for errors in _setup_task Zdenek Kabelac
2010-12-21 17:12 ` Milan Broz
2010-12-22 10:23 ` Zdenek Kabelac
2011-01-05 0:24 ` Alasdair G Kergon
2011-01-05 2:03 ` Alasdair G Kergon
2010-12-21 15:41 ` [PATCH 10/23] Fix memory leak of dev_filter on error path Zdenek Kabelac
2010-12-21 17:21 ` Milan Broz
2010-12-22 10:05 ` Zdenek Kabelac
2011-01-05 0:35 ` Alasdair G Kergon
2011-01-05 1:02 ` Alasdair G Kergon
2010-12-21 15:41 ` [PATCH 11/23] Add stack traces for archive Zdenek Kabelac
2010-12-21 15:41 ` [PATCH 12/23] Add tests for dm_task_set operations Zdenek Kabelac
2010-12-21 15:41 ` [PATCH 13/23] Make it obvious this code is not used Zdenek Kabelac
2010-12-21 15:41 ` [PATCH 14/23] Add stack trace for backup and backup_remove Zdenek Kabelac
2010-12-21 15:41 ` [PATCH 15/23] Add test check for find_config_str Zdenek Kabelac
2010-12-21 17:32 ` Milan Broz
2010-12-22 10:00 ` Zdenek Kabelac
2011-01-05 0:49 ` Alasdair G Kergon
2011-01-05 1:01 ` Alasdair G Kergon
2010-12-21 15:41 ` [PATCH 16/23] Remove check for vg != NULL Zdenek Kabelac
2010-12-21 15:41 ` [PATCH 17/23] Add check for dm_snprintf result Zdenek Kabelac
2010-12-21 17:38 ` Milan Broz
2010-12-22 10:34 ` Zdenek Kabelac
2011-01-05 0:12 ` Alasdair G Kergon
2010-12-21 15:41 ` [PATCH 18/23] Return PERCENT_INVALID for error case Zdenek Kabelac
2010-12-21 17:47 ` Milan Broz
2011-01-05 2:33 ` Alasdair G Kergon
2010-12-21 15:41 ` [PATCH 19/23] Warning - dead code problem elimination Zdenek Kabelac
2010-12-21 17:55 ` Milan Broz
2010-12-22 9:35 ` Petr Rockai
2011-01-05 2:15 ` Alasdair G Kergon
2010-12-21 15:41 ` [PATCH 20/23] exit if lvmchache_init fails Zdenek Kabelac
2011-01-05 1:22 ` Alasdair G Kergon
2010-12-21 15:41 ` [PATCH 21/23] Add default error path for get_property Zdenek Kabelac
2011-01-05 2:05 ` Alasdair G Kergon
2010-12-21 15:41 ` [PATCH 22/23] Remove dead assignment in dm_event_get_registered_device Zdenek Kabelac
2011-01-05 2:13 ` Alasdair G Kergon
2010-12-21 15:41 ` [PATCH 23/23] Remove dead assignment in vg_split_mdas Zdenek Kabelac
2011-01-05 1:59 ` Alasdair G Kergon
2010-12-21 18:09 ` [PATCH 00/23] Static analyzis set Milan Broz
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=cover.1292945707.git.zkabelac@redhat.com \
--to=zkabelac@redhat.com \
--cc=lvm-devel@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.