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 ';'

Suppress 'No PV label' message when removing several PVs without mdas.

Fix default /etc/lvm permissions to be 0755. (2.02.66)

post-release

pre-release

Fix _output_field crash from field_id free with DEBUG_MEM. (Phillip Susi)

The _free_vg that is created as a placeholder when reporting segments in pvs
was lacking the (vgmem) pool. We now create that pool. There is at least one
more such VG (_dummy_vg) which is pool-less. I am not sure what is the right
way to go about this, but this is currently necessary to fix a segfault
introduced by using vgmem in the reporter in Dave's lvseg lvm2app patches.

Signed-off-by: Petr Rockai <prockai@redhat.com>

Update interactive tests for lvm2app lvseg and pvseg apis.

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

Add lvm2app function to query pvseg properties.

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

Add the macro and specific 'get' functions for pvsegs.

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

Add a new type and function to lvm2app to enumerate pvsegs.

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

Add lvm2app function to query lvseg properties.

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

Add the macro and specific 'get' functions for lvsegs.

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

Add a new type and function to lvm2app to enumerate lvsegs.

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

Make value.string const char *, in properties.h, to fix a warning introduced by
the previous patch set.

Add vg_set_property to the interactive lvm2app test program.

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

Implement lvm_vg_set_property() by calling internal 'set' property function.

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

Add generic infrastructure to internal library to 'set' a property.
Similar to 'get' property internal functions.
Add specific 'set' function for vg_mda_copies.

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

Remove tag length restriction and allow / = ! : # & characters.

WHATS_NEW FIXME

Support repetition of --addtag and --deltag arguments.
Add infrastructure for specific cmdline arguments to be repeated in groups.
Split the_args cmdline arguments and values into arg_props and arg_values.

Preserve const for char pointer

Keep char pointers 'const'  (introduced with cling commit).

Add date configurable variable DATE

Follow the rule to run every command through variable dereference.
Add a runtime check of translated date to seconds.

fsadm fix for downsize of unmounted fs

Fix for the last commit as $MOUNTED is not only used as bool flag,
but also store mounted location for remount - so parsing output
from mount differently then from /proc/mounts.

Prefix calls of 'tunefs' tools with LANG=C to be sure we always do get
some nonlocalized strings.

Avoid using forced 'resize2fs' for cleanly unmounted filesystems and
run regular fsck -f for this case as required by resize2fs.

'fsadm check' uses date difference for extX filesystems between
the last mount and last check of 'fsck -f' execution and if the mount
was later run 'fsck' with -f so resize2fs is happy and user does not
need to pass '-f' flag.

Update fsadm regresion

Scan also 'mount' output for mounted filesystem.

As util-linux package seems to give all the time different names,
try harder to figure out, where is the given lv possible mounted
and scan /proc/mounts and if not found there, test also 'mount' output.

/dev/dm-xxx
/dev/mapper/vg-lv
/dev/vg/lv

All of them could be used different combination in /proc/mount and mount output.

Patch fixes regression for older systems where new detection code failed to
find valid combination.

Extend cling allocation policy to recognise PV tags (cling_by_tags).
Add allocation/cling_tag_list to lvm.conf.

Regenerate configure with 'autoreconf' for --enable-ocf. (2.02.76)

forgotten to regenerate last time configure.in was updated

post-release

pre-release

Fix include commit and switch to use DISTCLEAN_TARGETS

Fixing warning introduced by 'include make.tmpl' commit.
Produced this warning:

Makefile:29: warning: overriding commands for target `distclean'
../make.tmpl:366: warning: ignoring old commands for target `distclean'

Clarify error messages when activation fails due to activation filter use.

Add OCF support

Updated patch from Florian Haas from Linux-HA project.

User needs to 'configure --enable-ocf' to get file installed
by 'make install' target by default.
User can also use 'make install_ocf' to get only ocf files installed.
With disabled (default) ocf support - no ocf files are installed.

FIXME: ocf installation path needs to be kept in sync with pacemaker.
find better way and possible also better location.

Add given user prefix to make target install_initscripts

Avoid files to be written into the live system if lvm was
configured with different --prefix.

Use initdir for install target path.

Use include make.tmpl

Makes easier to use recursive targets and simplifies Makefile.

lost line

Fix regex optimiser not to ignore RHS of OR nodes in _find_leftmost_common.

Use new status code from fsadm check

Patch updates exec_cmd() and adds 3rd parameter with pointer for
status value, so caller might examine returned status code.
If the passed pointer is NULL, behavior is unmodified.

Patch allows to confinue with lvresize if the failure from fsadm check is
caused by mounted filesystem as many of filesystem resize tools do support
online filesystem resize. (originally user had to use flag '-n' to bypass
this filesystem check)

Add DIAGNOSTICS section to fsadm man page.

Return different status code for fsadm check of mounted filesystem

Return status code 3 for fsadm check of mounted filesystem - used later with
lvresize update patch to better support online filesystem resize.

Also makes a more consistent user interruption and returns status code 2
in this case.

Use dm_strdup/dm_free instead of strdup/free.

Allocate buffer for reporting functions dynamically to support long outputs.
Fix memory leak of field_id in _output_field function.

There's been a patch added recently to use dynamic allocation for metadata
tags buffer to remove the 4k limit (for writing metadata out). However, when
using reporting commands like vgs and lvs, we still need to fix libdm reporting
functions themselves to support such long outputs. So the buffer used in those
reporting functions is dynamic now.

The patch also includes a fix for field_id memory leak which was found in
the _output_field function.

Update VG metadata only once in vgchange when making multiple changes.
Allow independent vgchange arguments to be used together.
(Still more inconsistencies to iron out here.)

Update WHATS_NEW.

Add code to the dmeventd snapshot plugin to automatically unmount snapshots
that have been invalidated.

Alasdair correctly pointed out that if the two closes are concurrent (I haven't
checked, so they *might*), there is still a race possibility with the last
fix. This patch fixes that.

Add a missed test for dmeventd -R.

Update WHATS_NEW.

Fix a double close in clvmd.

The management threads (main_loop, the socket thread) could close a single fd
twice in a row sometimes. At least one other thread can be running at the same
time as the threads doing the double close. That one running thread also
happens to do some IO (namely, open /proc/devices, read from it, close it). If
there was enough "demand" for the local socket, this could happen:

- a connection to clvmd is about to finish, let's say the fd is 13 (it often
  happens to be in my test script, don't ask why)
- the local_sock thread calls close(13)
- the lvm thread calls open("/proc/devices"...) and gets 13
- the main_loop thread calls close(13) [OOPS!]
- new connection arrives, and is accept'd by a (new) local_sock thread
- the accept gives an fd of 13 (since it's the lowest free fd at this point)
- the lvm thread gets around to read from it's /proc/devices handle... 13,
  again
- the lvm thread hangs forever trying to read from the socket instead of
  /proc/devices

Signed-off-by: Petr Rockai <prockai@redhat.com>
Reviewed-by: Milan Broz <mbroz@redhat.com>

Use a more generous timeout for dmeventd in t-lvconvert-repair-dmeventd.

Let's rely on the exit code and do not test for empty output.

Add missing return for NULL passed buffer

Function pull_stateo() checks for NULL 'buf' - but return for this error
path was missing.  cmirror code never calls this function with NULL 'buf',
so this fix has no effect on current code base, but makes clang happier.

Macro uninitialized_var gives warnings in static analysis

Deactivate uninitialized_var() macro for clang static analysis.

Hotfix usage of __builtin_unreachable()

It's quite new feature which is not supported by older compilers.
So until some better macros are introduced into LVM code - hotfix current
compilation problems and compile this code only for __clang__ defining compilers.

Fix NULL pointer dereference for too large MDA error path

Replace dereference of NULL vg with passed vgname to the function
_vg_read_raw_area() in the error path for too large MDA.

Instrument compiler about code unreachability

Clang needs some instrumentation help for static code analysis.
It helps gcc and human reader as well.

Use static indentifier for internal functions

Functions _align_chunk() and  _new_chunk() are used only internally inside
pool-fast.c - so keep them static inside this object file.

Update C declaration  () -> (void)

Remove bufused for calculation

As bufused is assigned 0 in preceding source line
clang Idempotent operation

Fix vgchange to process -a, --refresh, --monitor and --poll like lvchange.

Simultaneous -a and --refresh is not valid.
poll+monitor are valid together with or without -ay* (but not with -an*)

No longer print polling results summary if no LVs in the VG were polled.

Explicitly have grep print the matching filenames (needed to
successfully determine scsi_debug device).

Update WHATS_NEW for lvm2app property functions

post-release

Add interactive tests for lvm_{pv|vg|lv}_get_property().

Add lvm_lv_get_property() generic function to obtain value of any lv propert

Add a generic LV property function to lvm2app, similar to VG function.
Return lvm_property_value and require caller to check 'is_valid' flag
and lvm_errno() for API error.

Add lvm_pv_get_property() generic function to obtain value of any pv property.

Add a generic PV property function to lvm2app, similar to VG function.
Return lvm_property_value and require caller to check 'is_valid' flag
before using the value.  If 'is_valid' is not set, then lvm_errno()
should be used to obtain the specific error.

Add lvm_vg_get_property() generic vg property function.

Add a generic VG property function to lvm2app.  Call the internal library
vg_get_property() function.  Strings are dup'd internally.
Rework lvm_vg_get_property to return lvm_property_value and require caller
to check 'is_valid' flag.  If !is_valid, the caller can check lvm_errno()
for the specific error.

Create a 'get_property' function, local to lvm2app, that factors out
most of the common code that copies the components of lvm_property_type
into lvm_property_value.  This allows for a 1-line function for each
of the generic property functions exported by lvm2app.

Add 'is_integer' flag into internal lvm_property_type.

Add 'is_integer' flag similar to 'is_string'.
Suggested in review by Petr Rockai.

pre-release

Use const config node

Fix constness warning

Fix usage of const 'data' pointer and also assign void* directly without
uneeded cast for C.

Fix constness warning

Keep using const pointers.