cov: replace strcpy with memcpy

When we know the size we can avoid using strcpy.

cov: use memccpy

When we want to copy string which may not be null terminated,
replace strncpy with more correct memccpy.

dmeventd: restart checks for running daemon

Check whether the pid file is associated with running daemon.
If not there is not much point to wait many seconds in loop.

toolib: skip when there is no vg and error_vg

We must skip even if the skip would be suggesting otherwise
if there is no vg and no error_vg as there is nothing to process.

configure: add --with-default-event-activaion=ON

Add new configurable option for building lvm2 with enable/disable
default autoactivation setting.

Might be useful for building i.e. rpms for systems where
this event_activation is not desired.

reporter: correcting failure return value

This function do use ECMD_ return values,
so use ECMD_FAILED as initial value - used for internal error paths

gitignore: add more ignored suffixes

makefiles: add libdm as deps for unit-test

When using just 'make check' ensure we make also dmsetup
so it's not needed to run explicit 'make' to get all the tools.

tests: include date and hostname IPs in test log

When our testing machines are going wild with NetworkManager,
this might be slightly helping.

tests: add timestamp to kernel line

Parse timestamps included in kernel kmsg line and add current system
time to the messsage as well - this makes it easier to look over,
when chasing some messages in journal after some time...

tests: check for supported dBus interface

Instead of blindly relying on sleeping, that dBus understands our
dBus interface, query the info with dbus-send.

tests: query dmeventd instead of sleep

Checking status of running instance of dmeventd should be a better 'sleep'.

dmeventd: check for running dmeventd for status

Before initiation fifo communication, check whether there is
running dmeventd - in case there is no running instance, this
would be just blocked for 5 seconds trying to open fifo.

tests: use non inlined copy constructor

gcc would like to see code without trying to inline large
copy constructor used for std::vector

makefiles: avoid using # with awk

Also awk got annoyed by this \# char sequence which is reported
as incorrect, however older rpm builder were failing without this.
So let's just try other variant.

makefiles: use SHOW

gcc: clear some complains

Use dm_strncpy() were possible to get rid of gcc compile warnings.

WHATS_NEW: update

tests: add sleep

mdadm may eventually be too slow...

pools: no checking for lv/vgremove deactivation

Thin-pool and cache-pool targets got already quite stable so let's try
to remove checking of pools when using lvremove or vgremove commands.

This skips checking pools when they are going to be removed, but it
also when removing thin volume that was the only user of a thin-pool.
In this case thin-pool will be still there and could be activated
again with another thin volume and thin_check will be executed
in this moment. In this case it can delay discovery of metadata damage.

thin-pool: avoid reactivation on new thin-pool

Shortens processing of 'lvcreate -L -V -T' command and
avoid deactivation and its activation with thin_check of the empty
created thin-pool that will be used for the new thin volume
made with a single lvcreate command.

vdo: modprobe dm-vdo for kernels >= 6.9

Modprobe "dm-vdo" with new kernels >= 6.9 and go for
'kvdo' with older Linux kernels.

WHATS_NEW: update

tests: skip convertions tests for usptream driver

tests: add correct condition for skip of lvmpolld tests

Check when flavour want to test  lvmlockd & lvmpolld tests and
in for case ignore SKIP_WITH_LVMPOLLD from test script.

vdo: use stats msg parser for also segment status

Shuffle code to parsing VDO message also for lvs segment status
so it can report correctly data usage for VDO LVs.

For this change move code and also change its API to use just mempool.

Fixes usage with upstream 6.9 vdo target driver.

vdo: fix status message parsing

When using message API for parsing VDO stats info, 0 was wrongly
used for fallback for trying the old sysfs API.

Switch to use  ULLONG_MAX for values that could not have been obtained
through the message call.

Fixes lvdisplay info for freshly created VDO volume with 0 used data
blocks.

tests: updated check for dm-vdo module

With new kernels dm-vdo is now correct name,
so use standard loading and just try extra for kvdo.

tests: inittest fix skipping condition

dev_manager: improve readability

Make a seperate function to decode which ID should be user
for cvol meta or data volume - also avoids duplication of code.
As a result it's now also easier to see how the lvid is build.

device_id: add comment for trim

lvmlocks: rework dm_strncpy

Fix cutting away 1 character via incorrect usage of dm_strncpy
introduced in last batch of commits and use sizeof(buffer) to
get proper sizes.

In case of use strcpy_name_len() the replacement was invalid,
so we need to restore this case since sanlock uses buffer without
nul ending, so we would strip one more character from the buffer.

Also start to use dm_strncpy() without (void) for unchecked usage.

refactor: update dm_strncpy to _dm_strncpy

For checked versions of dm_strncpy use inline wrapper _dm_strncpy.

refactor: remove (void) from dm_strncpy usage

Start to use dm_strncpy() as unchecked version within source tree.

util: add _dm_strncpy

Add internal inline function wrapper for dm_strncpy().
Use it for calls where we test the result.
Avoids emitting warnings in Coverity for unchecked usage.

util: swap header file load order

Ensure libdevmapper is included before util.h since
we are going to add function that is using it.
TODO: maybe add include directly here...

cov: missing initializer

Ensure there is no code path with uninitialized takes_arg.

lvmcache: free list of add_cache_devs

lvmdevices: fix memleak of use_old du

missed in the the prev done_old commit

cleanup: remove lvmcache_has_bad_metadata

cleanup: drop unused code

filtes: remove lvmetad bits from usable filter

Eliminate some dead code from usable filter, that is no longer usable
after removal of lvmetad.

device_id: simplier loop control

device_id: use dm_strncpy

Simplify code.

deviced_id: use zalloc

While moving to dm_strncpy, in this case since pvid is also
passed to crc calc, make sure whole buffer is always initilized.

filter: zero internal structure

Valgrind noticed access to unitialized warned_filter.

lvmdevices: fix memleak of done_old du

'done_old' list missed released.

cov: cast to uint32

cov: validate count is bigger then 0

To avoid teoretical underflow...

tests: better constructor

cov: validate major and minor

Check  major and minor is >= 0.

cov: store 64b flags

Alhtough no target is currently using that many bits for their flags,
preserve the size for loaded segment internally.

cov: validate i before underflow

Although it can't happen in this code, ensure 'i' is not 0
before decrementing it.

cov: remove unnecessary sanity check

This sanity check actually confused in some way Coverity
giving it some assumption about array access.

Since these two checks basically validated compiler's capability
to add and then substract the number from char pointer we likely
don't really need them - as if this base math would not work,
compiler would be having far more troubles...

So drop them - and get rid of report:
Event overrun-call: Overrunning callee's array of size 513 by...

cov: use dm_strncpy

Use better variant.

cov: no overwrite of const qualified fields

cov: reset FILE on error path

Although fclose() fails - we should not be retrying with
already released stucture.

tests: lvmdbusest.py faster test_z_sigint

Reduce #lv and intervals.

We are trying to ensure that the daemon stops while it's busy processing
its internal queues.  Decrease the number of LVs and intervals to allow
this test to complete in less time.

tests: lvmdbustest.py skip if g_tmo != 0

This test doesn't have any additional value when running when g_tmo != 0.

lvmdbusd: Exit faster

Add a global timeout value to be used for the threads to end waiting for
whatever it is they are blocked on.  The values varied from 2-5 seconds,
which is way longer than needed.  Value of 0.5 shows no CPU load when
service is running and is idle.

cleanup: drop unused code

Code was related to long time obsoleted  vgconvert
for lvm1 to lvm2 conversion.
This code likely was missed to be removed.

debug: update some structure members

Makes pahole output look slightly better.

commands: reduce command structure size more

Check for internal limits, if they would ever need to be raised report
error message and fail parsing.

cov: ensure buffer always ends with nul

When we print anything out of buffer, ensure there is \0.

cov: ensure nul ends string

tests: skip some mirror tests for polld and lockd

Leave just lvconvert-mirror-basic-1 test
for polld and lockd testing.

gcc: use uint16_t for counters

gcc: move declaration into ifdef

Prevent unused warning when the ifdef code is not compiled.

gcc: match signed integers

configure: lcov needs -fprofile-update=atomic

To avoid negative counters.
See: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68080

device_id: use dm_basename

Avoid problems for other libc like muslc and use dm_basename.

Prototype for basename has been removed from string.h from latest musl [1]
compilers e.g. clang-18 flags the absense of prototype as error. therefore
include libgen.h for providing it.

[1] https://git.musl-libc.org/cgit/musl/commit/?id=725e17ed6dff4d0cd22487bb64470881e86a92e7

Reported-by: Khem Raj <raj.khem@gmail.com>

dmsetup: refactor getopt_long usage

Shuffle code to avoid using static variable to pass parsed option.
Code is now easier to follow and also number of coverity reports
will go away.
There should be no functional change.

tests: missed reduction of raid check

cov: use proper enum value

cov: ensure namelist is defined

Seems coverity doesn't have good model for scandir.
Let's ensure pointer is seen as defined.

cov: validate return from display_lvname

Ensure strdup will not get NULL.

cov: validate origin pointer

Add internal error for NULL origin here.

cov: drop some unneeded continue

cov: check for syscall resuls

cov: ensure buffer is nul ended

Use dm_strncpy() to ensure ending nul.

tests: fix improper merge within last update

makefiles: ignore negative errors for lcov

tests: check for raid shrink support

Shrink of raid LVs supported from 1.9

cov: check for to_dlm_mode return value

Avoid processing invalid to_dlm_mode in lm_convert_dlm.

cov: check for strftime result

devices_id: add some syscall checks

Add debug tracing for syscall failures.
Also switch some log_error to log_warn when command does not exit
with 'error' result and only warns user.
Easier error path handling.
Initialize some vars at declaration time.

gcc: ensure num has always initilized value

cov: ensure string ends with \0

Use dm_strncpy() that ensures \0 termination.

cov: fix incorrect error check

Check for NULL pointer.
Fix missing release on error path.
Also dir_fd might be just != -1.

tests: update lvconvert-raid.sh

Handle case of removal of orig leg while it's being synchronized.

tests: update raid_leg_status check

Do not wait for progres (non zero processed amount of blocks),
when the raid status already reported 'resync' or 'recover'.

tests: aux avoid discarding created loop devices

Usually it's not a big deal, but couple test creates
large devices and here discard operation just excersice
CPU and consumes time.

tests: reduction needs to ignore fs

Drop 'should' usage and use  --fs ignore when reducing raid LV.

tests: update extend of raid

Here we actually need to slowdown only $dev2 - since repair operation
is only reading data from this device and compares it with origin $dev1,
and if they match there is no write...

commands: refactor memset

Move memset() to the initialization function define_commands().

There is also not much point in clearing memory on command's exit
so drop zeroing of ~2M of RAM.

commands: reduce structure sizes

Reduce slightly RAM usage of running lvm2 and use smaller fields
to count small numbers.

commands: use nul for EOL for compiled-in buffer

Use \0 as EOL in compiled-in syntax description to avoid
unnecessary line copy that just replaced \n with \0.
Also use already splitted lines when possible.

reporting: reuse existing _get_field

Use already existing _get_field() function that also
slightly optimizes 'strlen()' usage.

dev-cache: change locales just once

raid: update dm_get_status_raid

Handle mismatch of reported 'dm raid' status, where the active
raid LV can be actually showing higher numebr of raid leg devices,
that the number of shown status characters.

This can happen if the raid leg is dropped during initial
resynchronization.