Return PERCENT_INVALID for error case

If the percent value could not be determined return PERCENT_INVALID.
Indent function with tabs.

Remove unused variable mirr_state and its assignment

Remove dead assignment to thisfd

Value of 'thisfd' is not read again after its assigment

Fix another bug in t-pvcreate-operation-md (should pass now).

Add a test for RHBZ 640621 (metadata/dirs + no PV MDAs broken).

Fix typo in t-pvcreate-operation-md.sh.

Move the mkdtemp functionality into test/lib/utils.sh.

Fix a couple of bugs in the new (test/lib/)aux.sh.

Substantial rework of the functional test support code. Some new features:

- somewhat neater, more consistent and more readable output
- possible to set any lvm.conf value: aux lvmconf "section/key = value"
- LVM_TEST_NODEBUG to suppress the (lengthy) "## DEBUG" output
- back-substitution on test output ($TESTDIR/$PREFIX -> @TESTDIR@/@PREFIX@)
- support code moved from test/ to test/lib/ --> less clutter

Fix bug 635949: lvconvert man page clarification

Change to correct cmirrord return code is user visible - update WHATS_NEW
file.

Fix bug 632681: cmirrord shouldn't fail 'start' if already started

Return 0 from the cmirrord init script if the daemon is already
running.

http://refspecs.freestandards.org/LSB_3.1.1/LSB-Core-generic/LSB-Core-generic/iniscrptact.html

Export DM_CONTROL_NODE_UMASK and use it while creating /dev/mapper/control.

Use system assert in test/api/percent.c, for now.

Fix wrongly paired unlocking of global lock in pvchange. (2.02.66)

Remove check for existance of vg pointer

Checking for vg being != NULL in this place is not needed.
Pointer vg is already dereferced in this function above this code line.
Also this internal function _read_pv is always called with valid 'vg' pointer.

Add backtraces for backup and backup_remove fail paths

Hide unused code into if 0

Make it obvious for lcov coverage and static analyzis we
are not interested in this piece of code.

Detect errors from dm_task_set calls

Check for errors in dm_task_set calls.
Use  goto_bad macro with stack trace.
Replace  label  failed: with bad:.

Fix last pthread_join fix commit

Call for pthread_join() does not set errno value even though return values
looks like that. For now assign errno from return value and still use
strerror() to print some error message as this seems to be commonly used.

Add also log_sys_error() message for error close of local pipe.

Add backtraces for archive and backup_locally

If archive or back_locally fails - add stack trace.

Fix memory leak in debug mode of restart_clvmd() error path

Log error state from pthread_join operation

Value jstat is unused - so replace it with logging via log_sys_error().

Remove dead assignment of lv_total and lv_capasity_total

Variables 'lv_total' and 'lv_capasity_total' are unused.

post-release

Copyright notices for new files.

pre-release

Add missing test for reallocation error.

Remove dead assignment

Variable 'r' is never read so remove it and just cast result from
_error_device function to (void).

Verbose log old_umask value

Use old_umask value and print its content through verbose log.

Add internal error if pointer is uninitialized

Add simple check for existance of 'pl' and printer internal error message
if device is missing instead of plain crash.

Add check for unlink errors

Remove unused variable label

Variable 'label' is unused in _format1_pv_write().

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.)