Remove unused variable dev_dir_provided

Remove dead assignment of segh

Variable 'segh' is never read again after this assignment.

Remove dead store in lvm_run_command

Variable 'ret' is not read before its next assignment.

Removed unused pointer

Pointer 'duplicate' is unused.

Use dm_free for dm_malloc-ed areas in _clog_ctr/_clog_dtr (cmirrord).

Use dm_zalloc to obtain zeroed memory block.
Use dm_free for dm_ allocated memory blocks.
Test close() for error.

Add checks for allocation errors in config node clonning.

Add checks for clonning allocation a fail-out when something is
not allocated correctly.

Also move var declaration to the begining of the function
and fix log_error messages.

Replace multiple fprintf calls with one large string parameter

Fix error path if regex engine cannot be created in _build_matcher().

Fix only 'stack' printing with full function error exit.

Use const char * for name and old_name in vg

Switch to use const char pointers to avoid changes of these structure
members and having better control over, were these members could
be modified.

Use const char* for offset calculation

As 'const' types are also passed to macro dm_list_struct_base -
keep offset calculation with const char pointers.
Fixes several gcc constness warnings.

Switch void* to char* arithmetic

Remove const usage from destroy callbacks

As const segment_type or const format_type are never released
use their non-const version and remove const downcast from dm_free calls.
This change fixes many gcc warnings we were getting from them.

Fix wrong cast to char*

As cmd->cmd_line is already const char pointer it's not needed to cast already
const char pointer to char pointer.

Use const char* const * for dm_regex_create()

Change API interface to accept even completely const array patterns.

This should present no change for libdm users and allows to pass
pattern arrays without cast to const char **.

Some const cleanups

Minor const warning fixes and internal API updates.

update

Test return value from read() and close() for an error.

Add more strict const pointers around config tree

To have better control were the config tree could be modified use more
const pointers and very carefully downcast them back to non-const
(for config tree merge).

Move var declarations to function begining

As assert macro jumps to 'bad:' label - we need vg initialized.

Fix NULL pointer check for *buf

As ternary operator has lower priority then add operation, this check
was not doing what seemed to be expected.

So enclose the test in braces and check for NULL in *buf.

Fix device.c #include to ensure 64-bit fopen64 use. (2.02.51)  (robbat2)

Fix sed substitution in copying tests to builddir (test/Makefile.in).

Add getters for copy_percent and snap_percent to the lvm2app API.

Add further consistency checking to vg_validate, ensuring that all segment
areas point to LVs or PVs that are listed in the respective VG.

Hack up the RUN_BASE computation in the test Makefile.in a bit more (so that it
actually works... sometimes).

Add a validation step for pvmoveN internal LVs to vg_validate.

HAVE_SELINUX again

#ifdef HAVE_SELINUX and #ifdef HAVE_SELINUX_LABEL_H

Missing #elif HAVE_SELINUX

Update configure.

Create /var/run/lvm directory during clvmd initialisation if missing.

We need to be sure that /var/run and /var/lock is always there.
(E.g. these two directories could be using tmpfs which then loose
all the content after reboot.)

Add new dm_prepare_selinux_context fn to libdevmapper and use it throughout.
Detect existence of new SELinux selabel interface during configure.
Use new dm_prepare_selinux_context instead of dm_set_selinux_context.

We should set the SELinux context before the actual file system object creation.
The new dm_prepare_selinux_context function sets this using the selabel_lookup
fn in conjuction with the setfscreatecon fn. If selinux/label.h interface
(that should be a part of the selinux library) is not found during configure,
we fallback to the original matchpathcon function instead.

Remove a redundant %.o: %.c rule from test/api/Makefile.in.

Fix the sed expression to get "base" names of tests.

Only build the required bits under api/ on make check.

Do not build vgtest (superseded by vgtest.t). Fix the build line for .t files.

First go at a somewhat more comprehensive mechanism to run "unit" tests for the
lvm2app API. Further factoring of the support code needed. RHBZ 654445

Fix scanning of VGs without in-PV mdas.

Set cmd->independent_metadata_areas if metadata/dirs or disk_areas in use.
- Identify and record this state.

Don't skip full scan when independent mdas are present even if memlock is set.
- Clusters and OOM aren't supported, so no problem doing the proper scans.

Avoid revalidating the label cache immediately after scanning.
- A simple optimisation.

Support scanning for a single VG in independent mdas.
- Not used by the fix but I left it in anyway as later patches might use it.

Try to detect fail in clvmd startup in tests.

.

.

Rename vg_release to free_vg.

Cope better with an undefined target_percent operation in _percent_run.

Remove reset of vg->vgmem pointer as it is access of already release memory

This reset of vgmem pointer causes access of already released memory.
(_vg_make_handle allocates vg from vgmem pool itself - which is a bit tricky)

Interestingly this memory fault was missed by our test suite.

post-release

pre-release

Fix debug logging of derived flag LCK_CACHE in clvmd.

Check str_list_add() success

Report error if str_list_add fails.

Check lv_info() success

Add log_error message for lv_info failure and exit from futher
processing.

Replace 'leg' occurence in debug message with 'image' which
is used in other messages.

Add backtraces for errors

Add stack;  backtraces when error is reported from dev_set() or
dev_close_immediate().

Log error from unlink failure

Test lv_name is not NULL

Patch adds extra check for lv_name not being NULL.
Test avoids unneeded strlen call for this case.
Otherwise there is no functional change as test would fail on
size_t comparation even for NULL lv_name (thus there is no risk
of NULL dereference when taking 'true' if branch.

Add logging for pipe write() and close() error

Check values from write() and close() system calls.

FIXME: Missing wrapper around 'write()'.

Check result of vginfo_from_vgname

Check for some potential internal error.

Optimize second call to strchr with same parameters

Small optimalization - reusing already known strchr result.

Fallback to full rescan for missing device

Fix bug when NULL could have been passsed as 'data'
to _add_pv_to_list() if 'dev' is NULL.

Now it fallbacks to complete scan.

What's new.

Remove unneeded test for NULL

Remove check for system_id (it is defined as int8_t[], so cannot be NULL).

Test uuid for NULL

Add test for NULL before passing uuid as src argument to memcpy.
As memcpy function is declared as function not accepting NULL.
Though we pass NULL only with zero length so this patch presents
no functional change to the code.

Add stack trace for error path

If dm_task_set_cookie() fails print stack trace, but keep going on.

Add error path stack traces

Check for errors from dm_task_set_name() and dm_task_run().
Add stack traces for error paths.
Return 0 if some error is found.

Remove check for lv is NULL

'lv' is deferenced in the begining of the function so any check
later is not helpful.

Parameters for dev_manager_transien() are marked as nonnull.

Add missing test for failed pool allocation

Add test for NULL from dm_poll_create.
Reorder dm_pool_destroy() before file close and add label out:.
Avoid leaking file descriptor if the allocation fails.

Replace snprintf with dm_snprintf

Use dm_snprintf with known error case return code (-1).

Check reallocated buffer for NULL before use

As *buf is reallocated in case CLVMD_CMD_TEST: test for NULL is needed
before printing status.
(realloc() == NULL and status != 0)

Refactor the percent (mirror sync, snapshot usage) handling code to use
fixed-point values instead of a combination of a float value and an enum.

Fix a failing test (it used a combination of lvconvert parameters that is no
longer permitted.)

Avoid the automatic MISSING_PV recovery path in commands with special
MISSING_PV handling (cmd->handles_missing_pvs is set).

Fix memory leak when VG allocation policy in metadata is invalid.
Ignore unrecognised allocation policy found in metadata instead of aborting.
Fix another missing vg_release() in _vg_read_by_vgid.

Optimize lookup table read

Reread lookup table only when needed.

Remove dead assignment in wait_for_child

'pid' is not used anywhere - remove it.

Remove dead assignment in _step_matcher

'ns' is not used after this assignment and it is reassigned with the following
code, so dropping this assignment.

Remove dead assignment in _mirror_emit_segment_line

Remove unused 'r' assignment.

Fix memory leak in error path

Nicely hidden memory leak in outf macro error path.
This macro is using out_text() and does automagical return_0.
That would leak tag_buffer allocated memory.

As there was same code for tags output - create _out_tags() function.

Use one fprintf call for usage print

Replace multiple fprintf calls with multiline one.

Remove dead assignment in dm_tree_node_add_mirror_target_log

'seg' is never used - remove it.

Remove dead assignment in 'main'

'ret' is never read anywhere - remove it.

Remove unused 'i' in _pv_analyze_mda_raw

'i' is unused in the function - remove it.

Remove dead assignment in lvm2_main

'alias' is not read again in this code path.
Also 'alias' is already equal to 0 in this place.

Remove dead assignment in _lock_for_cluster

'saved_errno' is not read from this initialization and before its
usage is assigned again before _cluster_free_request() call.

Reset vg pointer after release

Set vg to NULL after releasing it as the following memlock() test may
lead to goto for the second call of vg_release() with the already
released vg pointer.

Remove printing of LCK_CACHE

LCK_CACHE is defined as 0x100 so it cannot be passed through
unsigned char parameter - remove it from the sprintf code.

If the LCK_CLUSTER should be printed here - lot of code need
to be reworked - so adding FIXME comment.

Fix check for empty system_dir

Fixing check for zero length system_dir string.

Cleanup remove test for NULL

dm_free is testing NULL itself

Fix "it's" typo to be "its" in lvconvert error message.

Update WHATS_NEW.

Disallow certain lvconvert operations that need to both allocate and free
extents, while physical volumes are specified. Fixes BZ 640051.

All 'size' values of lvm2app properties should be in bytes.
Fix 'seg_size' to return bytes.

Signed-off-by: Dave Wysochanski <wysochanski@pobox.com>
Reviewed-by: Petr Rockai <prockai@redhat.com>

Add interactive tests for functions to lookup a pv|lv by name|uuid.

Signed-off-by: Dave Wysochanski <wysochanski@pobox.com>
Reviewed-by: Petr Rockai <prockai@redhat.com>

This patch adds helpers to allow users to lookup a lv or pv handle by
uuid (given a vg_t of course).

Signed-off-by: Dave Wysochanski <wysochanski@pobox.com>
Reviewed-by: Petr Rockai <prockai@redhat.com>

This patch adds helpers to allow users to lookup a lv or pv handle by
name (given a vg_t of course).

Signed-off-by: Dave Wysochanski <wysochanski@pobox.com>
Reviewed-by: Petr Rockai <prockai@redhat.com>

Add missing destrustion of cmd_context

Lvm1 fallback code missed to destroy cmd_context in error path.

Fix memory leak in error path

Release allocated path buffer in error path.

Fix resource leak of dlopened pointer

Add missing dlclose in _init_formats() error path.
Use return_0 to print stack trace from the call.

Add missing fclose

Fixing resource leak in _umount().

CHECKME: mountpoints with spaces need to be checked

Move arg_vgnames from local scope

As gcc puts probably all vars on stack this bug was not noticed in runtime.
Patch fixes referencing local scope list variable.

Do not call dm_task_destroy with NULL

Add missing  closedir() - fixes resource leak

Move va_end() so it is also used before error path  return

Move va_end(ap)  so we do not leave with return -1 without calling it.
Remove unneeded ';'