Ensure allocated device does not leak on error path

For unimplementd canonicalize_file_name set to NULL

Add internal error for unsupported code paths

Patch mainly helps static analyzers to better work with code paths
lvm code should never trigger.

Check for missing reply_uuid

Check for allocation failure

Add check for allocation failure

Add few missing allocation failures tests

Add check for rimage name allocation failure

Add check for mda_copy failure

Add check for failure

Add free_orphan_vg

Move commod code to destroy orphan VG into free_orphan_vg() function.
Use orphan vgmem for creation of PV lists.
Remove some free_pv_fid() calls (FIXME: check all of them)
FIXME: Check whether we could merge release_vg back again for all VGs.

If the same fid is already same avoid ref_counting

Add missing test for failure of lvmcache_foreach_pv

Clean error paths for format instance

With updated orphan VG code this code needed some updates.
Add missing log_error for allocation failures.

Release_vg instead of plain free in error path

Log error reporting for failing _alloc_pv

Drop unneeded zeroing of zalloced memory region.

Do not write to -1 buffer address

In case of zero bytes would be read from sysfs, it would store '\0' on
temp_buf[-1] address.

Simplify some buffer length calculation and use strcpy if we've just
checked string fits in give buffer.

Replace jump label error: with bad: commonly used in libdm.

Remove duplicit test

When it's space it's also not a '\0'.

Update autoconf.

In the new corosync world, dlm is a standalone service.

Fix clvmd init script to Require dlm service when building for the
new corosync or clvmd will fail to start.

post-release

pre-release

FMT_INSTANCE_VG is redundant now

FMT_INSTANCE_PV is no longer used

use stack consistently if 0 is considered an error

missing error mesg

remove unused bits after fid changes

give standard error message if lstat fails unexpectedly

Fix message check

Check pointer from strchr for NULL instead of crash later.
Badly formated message would have crash dmeventd otherwise.

Check for deps pointer before dererence

As _deps() call may return NULL - check for it.

Add validation of name and uuid

Do not accept NULL pointers.

Do not crash for NULL sort_key

Guard against NULL pointer for sort_key and let it behave like an empty
string would have been passed in (i.e. no key).

Return error for failing allocation

Fix case, where final strdup would have failed and it would miss to return
failure for this case and return success and NULL pointer.

Add test for failing allocation

Avoid memcpy to NULL if realloc fails.

Add test for memory allocation failures

Replace asserts with test for failing memory allocation.
Add at least stack traces.
Index counter starts from 1 (0 reserved for error), so replacing fingerprint.

Remove unreachable code

What's new.

Keep a global (per-format) orphan_vg and keep any and all orphan PVs linked to
it. Avoids the need for FMT_INSTANCE_PV and enables further simplifications. No
functional change, internal refactor only.

What's new: lvmcache.

Move lvmcache data structures behind an API (making the structures private to
lvmcache.c). No functional change.

Stop processing lvextend if trying to extend a mirror that is being recovered.

Missing correct return value in lv_extend fn.

Move label_init up in code stack

label_exit() is called destroy_toolcontext() and we are now
using standard dm_list function for destroy, we have to make sure
dm_list gets initialized properly.

Thin add pool_below_threshold

Test both data and metadata percent usage.

A bit more readable code

Just a minor readability conversion.

Fix test for lv_snapshot_percent

Do not check for PERCENT_MERGE_FAILED if the lv_snapshot_percent() failed.
(test for snap_percent would be testing uninitialized value).

Drop unreachable code

Use dm_asprintf to simplify code

Some fixmes

'len' calculation is unused ?
Unreachable code could be removed or moved upward ?

Switch to return void

List delete cannot fail, so there is no reason to test for error.

Use dm_snprintf and improve error handling

Add standard error reporting with error logging.
Use plain alloc instead of zalloc for string buffer.
Use dm_snprintf with valid test for <0.

Add range test for device number

Check the output of atoi is in valid range.

Remove duplicate test

Tested condition has been already evaluated before
For strlen() code has already excluded <ID_LEN.
For repairing, already tested (!argc && !repairing) before.

Replace snprintf with dm_snprintf

snprintf testing for negative is replaced with dm_snprintf where this
test really works.
Add missing test for result of dm_snprintf().

Remove unneeded assignments

Variables have (or will have) those values set.

Keep page_size as signed number

Since it's return value from sysconf and is checked for <0.

Move done jump lower

Since before 'goto done' is bufused zeroed, it would otherwise write 1
byte in front of buffer.

Add boundary test for number of mirror devs and logs

As atoi may return negative value - test for both limits.
Test log_args for limits before calling alloca().
Code from dmeventd mirror plugin should probably share same code as
we have in mirrored.c.

Set all parameters to 0

Since the function dm_get_next_target() returns NULL as 'next' pointer
so it's not a 'real' error - set 0 to all parameters when NULL is
returned because of missing head.

i.e. one of use case::
do {
next = dm_get_next_target(dmt, next, &start, &length,
  &target_type, &params);
size += length;
} while (next);

Check that whole locking_dir fits _lock_dir buffer

Move close few lines

Since the function dev_close() has code path, which really could close
file (for unlocked vg) and destroy dev handler, stay on safe side and move
the close few lines later, even our current use case shouldn't trigger
such scenario.

Use dm_list_iterate_items_safe

And avoid direct access to list member variables.
Inline _free_li().

Fix fd resource leak in error path

Use 'goto bad' to cleanup fd on error path.

Ensure strncpy() function always ends with '\0'

Since last character needs to be \0 for string,
pass buffer size smaller by 1 byte.

Set status for error path

Do not leave status unitialized, since in some cases, it's tested,
when the function returns error.

Add missing deps for lvm2api

Hmm, wasted some time because of this missing deps....

Fix resource leaks for failing allocation

In case, something would fail during format initialization,
return allocated memory.

Release allocated resources in error path

If composite_filter_create() fails, release filters.

Check result of lstat

If lstat returns errno different from ENOENT, do not use the content of
struct stat 'buf'.

What's new.

Remove a dubious log message ("another thread is handling an event") from LVM
dmeventd plugins. Fixes RHBZ 771419.

post-release

pre-release

Adjust comments

Add synchornization point in mirror log init.

Put extra sync point when mirror log is deactivated and before
it's activated for the second time.

Disable partial activation for thin LVs and LVs with all missing segments

Count number of error and existing areas and if there is no existing area
for the LV avoid its activation.

Always disable partial activatio for thin volumes.

For mirrors currently put in hack to let it pass with a special name
since current mirror code needs to activate such LV during some operations.

Avoid warning for small pv_min_size

Do not print warning for pv_min_size set in range between 512KB and 2MB.

Clean up systemd unit ordering and requirements.

Minor consistency update for debugging messages

Use mlock/munlock for debugging lines.

User correct base dir for lcov reports

Fix problem when srcdir != builddir.

lost line

Track unreserved space for all alloc policies and then permit NORMAL to place
log and data on same single PV.

lp.pool may be (null) too

Automatically detect whether corosync clvmd needs to use confdb or cmap. (fabio)

Fix data% reporting

For reading % of mapped size of thin volume use as origin for
old style snapshot '-real' device needs to be queried.
Fix log_error report given for lvs -a in this case.

TARGETS macro has to be defined before inclusion of make.tmpl

Use @TESTING@ check in this case.

post-release

Build test run target only for $(TESTING)

Add libraries properly in LDLIBS.

Thin  for_each_sub_lv

Adapt to scan thin dependency LVs

If clvmd is not started, die test instead of skipping it

Oops missed braces in previous commit

This has disabled clvmd for being executed.
(FIXME improve testing part to catch this fault)

pre-release

pre-release

Cosmetic change use return macro with implicit stack

Set missing header define

Fix leak of hash table

Minor leak on command initialization.

Instrument code that pointer are already released

Set pointers to NULL since on the function exit they are no longer valid.

Set to a defined value vars used after error path

Static analyzer noticed this vars are used even when error is reported
back thus their state is undefined - set to 0 for this case.

Test for uname result

in fail path initialize to 0.

Fix missing dmt destructor

Also always initialize maj,min,patchlevel when success is returned.

Free allocated segment type in error path

Limit alignment to 32bit values

to get the same behavior on 32/64 machines.