raid: ensure area_count is at least 2

Enusure we will not divide by 0.

libdm: put in secure check

Coverity complains about NULL deref - while this cannot currently
happen, put in secure INTERNAL_ERROR.

reporter: add missing stack trace

Use goto_out on error paths.

lvmetad: tail chasing to shut up coverity

dmeventd: fix check for failing open.

Recent change 2c8d6f5c90d5be62b48ba2881f2a6631091dc5af
actually droped restart when the reason of failing open is missing
device completely - check for ENOENT now as another reason
to start new dmeventd server  (when there is no systemd to maintain it).

str_list: do not support str lists without mempools

Do not support str lists without mempools. Instead, create temporary
mempool where necessary (currently only _get_report_options fn).

configure: check for udev_device_get_is_initialized is available

The udev_device_get_is_initialized is available since libudev version
165. Older versions are still used somewhere (e.g. RHEL6). So better
check for this fn and use it only if it's available.

dev-ext: issue error if external_device_info_source=udev and udev db record incomplete

Udev db records are marked as not initialized (incomplete) on timeout.
Issue an error message whenever LVM finds such records so users are
aware that something's going wrong with udev db.

This is important in case we use devices/external_device_info_source="udev"
where udev database records are used to do various filtering decisions.

For example:

udev log of timed out worker:

Nov 11 13:02:25 raw.virt systemd-udevd[607]: seq 1997 '/devices/virtual/block/dm-2' is taking a long time
Nov 11 13:04:25 raw.virt systemd-udevd[607]: seq 1997 '/devices/virtual/block/dm-2' killed
Nov 11 13:04:25 raw.virt systemd-udevd[607]: worker [11221] terminated by signal 9 (Killed)
Nov 11 13:04:25 raw.virt systemd-udevd[607]: worker [11221] failed while handling '/devices/virtual/block/dm-2'
...

LVM also issues error message visibly if incomplete udev db record is found,
devices/external_device_info_source="udev" is set:

$ pvs
  Udev database has incomplete information about device /dev/dm-2.
  Failed to get external handle for device /dev/dm-2 [udev].
  ...

libdm: reorder error path

Coverity noticed recent fix of an error path missed to
release 'dmt' - reoder code to ensure 'dmt' is released.

raid: fix the string compare

Coverity noticed this condition is always false and the error
path could never be visited.

So check for all mismatches of supported messages
and actually mark log_error as internal error.

coverity: Add placeholder modelling file.

libdm: replace assign with increment

Coverity didn't liked assign with && expression, so use trick.
It does not complain against this prefix incremenent operation.

vgrename: check if new and old names match

When the first arg is a UUID and vgrename translates
that UUID to a current VG name, the old and new VG
names are not being checked for equality.  If they
are equal, it produces an internal error rather than
a proper error.

lvmetad: change recent cleanups

with better alternatives.

lvmlockd: reverse some unnecessary checking

These are unnecessary or not useful.

tests: handle missing delay_dev

Try to run as much of test which could run without delay_dev.

lvmlockd: fix the NO_GL_LS condition

indicating when no global lockspace exists.

cleanup: ensuring string is not NULL

Coverity cannot see the string cannot be NULL so make it explicit.

cache: ensure there is no NULL str

Coverity is not smart enough to detect this case could never happen.

lvmlockd: enforce 64bit arithmetic

Coverity suggest to stay on the 'safe' side and widen operators early
so 64bit offset is made from 64bit arithmentic.

lvmlockd: add missing error checks

Detect error from function and report them.

cleanup: use _ for local function

cleanup: drop unneded headers

Coverity notices they are not really needed.

dmsetup: one missed dms

libdaemon: fix passing 32bit values for %d

Since %d is now prohibited for its great confusion,
replace it with  FMTd64 and correctly converted int64_t
parameter.

lvmlockd: shut up warnings

libdaemon: prohibit use of %d for 64bit numbres

Do not let pass %d and print internal error when found one.
Require all users to use  FMTd64 - it's seriously confusing.

lvmetad: restore use of FMTd64

daemon_reply_simple() eats just 64bit numbers - so make it explicit.

tests: update test to check for race better

Use  delay_dev to slow down mirror sync so we could more
easily check for race and proper reject of parallel mirror
leg addition/reduction.

Also expose fail in mirror allocation of parallel leg.

tests: do not skip test when delay is missing

Rather then skipping whole test - just do not use it.

Failing tests that have required delay need to deal with reality
and shell either check for HAVE_DM_DELAY and skip portion
of test or using should when needed.

cleanup: update comment

dmsetup: cleanup warn for older compilers

Older gcc somehow thinks there is a path of using
subcommand uninitilized.

So make it more obvious there is no such one.

lvmetad: maintain seqno as int

Keep seqno as 32bit value.
Also use '_' prefix for local _update_metadata.

raid: mark intententional copy and paste

Coverity: add this extra comment, to let Coverity know this
slightly changed copy&paste code is intentional.

cleanup: use code in place

Pass const strings to printf(),
and use  struct names directly instead of creating unused vars on stack.

cleanup: reoder rstatus init

In all error paths always initialize *rstatus.

cleanup: coverity quiet

While through all codepaths we never 'read' lock_id unless LCKF_CONVERT,
coverity cannot decrypt this.

As since it's usually better to pass in 'well-defined' data structures
preset lock_id to 0.

cleanup: use fputs for plain strings

Use fputs() when printing plain string,
easier then fprintf which needs to parse it.

Also check fd before close is >= 0 -
it is - but coverity fail to see it, so eliminate
this false-positive warning.

cleanup: drop unneeded assign

prio is always assigned later.

cleanup: keep using enum typedef

Using enum instead of unsigned.

cleanup: use 64bit multiply for print

cleanup: drop unneeded header file

cleanup: use display_lvname

cleanup: use NAME_LEN

Let's have instant check for max name len when creating
subLV name.

cleanup: use NAME_LEN buffer instead of alloca

Drop alloca and use NAME_LEN size to get implicit check
for max len of LV name.

cleanup: do not test alloca for NULL

alloca() never returns NULL.
In case stack is out-of-range the behaviour is undefined.

cleanup: drop unneded assign

Coverity doesn't really like to see subcommand being assigned NULL,
so drop it.

alloc: use own mem pool for alloc_handle

Keep alloc_handle's data in a single mempool and do not
spread them into vgmem pool.

cleanup: relocate error capture

Capture internal error before allocation anything.

toollib: add missing check for lvmcache_init()

Coverity notices lvmcache_init() may fail so check and
error out in case of failure.

devmanager: validate target params

Coverity: ensure we do not read through NULL pointers for
target_type and params.

report: add extra pointer check

Coverity was seeing possible trouble with NULL pointer dereference.
So ensure it may never happen.

lv_manip: do not deref NULL for debug message

Coverity: when 'pv2' would be passed as NULL, do not try to
deref it in debug message.

dev-type: fix TOCTOU order

Doing 'stat' checking first and later opening is racy.
And since we do not really care about any 'status' info
here and we read 'sysfs' here - just drop whole 'stat()'
call and directly handle error from failing 'fopen()'.

thin: fix error path mem leak

Coverity: when parsing of thin-pool status would have failed,
it could have leaked memory pool and dmt struct.

libdm: add test for  dm_task_get_message_response()

Coverity notices dm_task_get_message_response() result should be
checked for NULL which should not be passed to dm_pool_strdup().

libdm: exlicitly check for NULL

Coverity: another explict check for NULL, where coverity fails to
see it.

libdm: ensure vars are initialized

Coverity found potential error path, where code could
have used some unset variables.

libdm: add missing error path check

Coverity: do not continue with section cloning when root node
would a NULL.

lvconvert: add extra check for existance of pointer

Coverity here is not fully-in-picture - but please it
with validation of pointer which currently cannot be null,
since we always return at least empty string.

lvmetad: validate mda is not NULL

Coverity: make it explicitely obvious metadata area is not NULL.

lvmetad: check for pointers not NULL

Check for  arg_vgid_lookup and arg_name_lookup not being NULL.
Drop checking arg_vgid and arg_name for NULL since they
are already dereference earlier - thus mostly must be NOT NULL.

(If that would be possible larger rework of this function would be
required).

lvmetad: require meta_lookup

Coverity: do not call update_pvid_to_vgid() with
meta_lookup == NULL since it is dereferencing it.

lvmetad: check for allocation fail

Coverity: missing check for root != NULL as this pointer is
later dereferenced in add_last_node().

dmsetup: add missing checks for dm_stats_create() ret value

Coverity likes to see a check for dms not being NULL, so
add those missing ones...

dmsetup: check for NULL from dm_task_get_ioctl_timestamp

Coverity: ensure NULL is not passed to dm_timestamp_delta().

clvmd: check for pthread_create status

Coverity: likes to see checked function result.

dmeventd: open fifo in one function

Put calls related to fifo opening into a single function.

Fix  Time-Of-Check-Time-Of-Use and use fstat()
and fchmod() on already opened fd instead of
checking first path and then risking to open something
different.

dmeventd: reorder tou

libdevmapper-event: fix fifo leak on error path

Coverity: when _init_client() fails, client fifo could have
been already openned and needs to be closed on error path.

configure: report whether to build lvmlockd

Report configured status for build with lvmlockd.

post-release

pre-release

test: disable duplicate VG name test for lvmetad

Until we decide how duplicate VG names should be
handled by lvmetad, and implement that.

mirror: Fix log size calc when more than 1 extent.

Currently the code creates the log separately after allocating space for
the data and as no data allocation is needed this second time,
total_extents ends up holding zero so use new_extents directly instead.

vg_read: skip repair and wipe for foreign and shared VGs

When reading a foreign VG we cannot write it, since
it belongs to another host.  When reading a shared VG
we cannot write it because we may not have an ex lock.
(Or we may be reading the shared VG while not using
lvmlockd in which case it's like reading a foreign VG.)

Add the same checks for wiping outdated PVs.  We may
read a foreign or shared VG, or see the PVs, while
another host is part way through writing a new version
of the VG to the PVs.  This might cause us to think
some of the PVs are outdated.  We do not want to
write another host's PVs, especially when we may
wrongly conclude they are outdated.

lvmcache: change log_verbose to log_warn

Without this, some cases miss printing a
warning for duplicate PVs.

lvmetad: log duplicate PVs returned from lvmetad

When the command gets a list of alternate devices
from lvmetad, log each one directly.  This is not
the same as the warnings when adding lvmcache,
which are related to which duplicate is preferred.

tests: skip duplicate vg names test with lvmetad

If two PVs have different VGs with the same name
(different uuids), one of the VGs is ignored by
lvmetad.  A FIXME exists in lvmetad to find a
better response.

lvmetad: refactor and document

update_metadata and pv_found update the cached metadata;
these are both reworked to improve the code, organize it
by each possible state and transition, make it much more
clear what's changing, add more error checking and
handling, and add comments.

The state and content of the cache (hash tables) does not
change (apart from some things that didn't work before),
and the communication to/from commands does not change.
The implementation and organization of the code making
the state changes does change significantly.

One detail related to the content of the cache does change:
different hash tables do not reference the same memory any more;
the target values in each hash table are allocated and freed
individually.

config: fix copy error in examples

The same example was copied without
changing the variable name.

test: api subdirectory is needed in PATH by pytest

test: Run pytest with installed libs

spec: Add python bindings

post-release

pre-release

WHATS_NEW: reporting commands and -o-/-o+ extension

commands: update command help for -o|--options for reporting commands

man: pvs/vgs/lvs and -o+, -o-, -o#

tests: add report-fields.sh test

report: support "-o #field_name1,field_name2,..."

report: recognize known prefix when processing "-o -field_name1,field_name2,..."

report: add report_get_field_prefix function

report: support "-o -field_name1,field_name2,...."

report: make report options defined by "-o" groupable

Also, besides making "-o" groupable, use string lists to store
lists of options temporarily while processing all instances of
the "-o" group.

refactor: move code detecting report options to a separate function

str_list: add str_list_destroy function

The str_list_destroy function may be called to cleanup memory when
the list is not used anymore and the list itself was not allocated
from the memory pool.

str_list: add str_list_to_str and str_to_str_list functions

The str_list_to_str and str_to_str_list are helper functions to
convert string list to a single string and vice versa.

str_list: also allow memory allocation without memory pool

New entries for pvs related fixes.