tests: get.sh declare and assign separately

Declare and assign separately to avoid masking return values.

Also comment-out currently unused  orig variable.

tests: get.sh double quote

tests: utils.sh separately assign

Declare and assign separately to avoid masking return values.

tests: utils.sh avoid iterating over ls output

tests: utils.sh arithmetic

expr is antiquated. Consider rewriting this using $((..)), ${} or [[ ]].
$ is unnecessary on arithmetic variables.

tests: utils.sh no mixed string array

Argument mixes string and array. Use * or separate argument.

tests: utils.sh double quote

tests: lvm-wrapper.sh legacy syntax

tests: lvm-wrapper.sh double quote

tests: inittest.sh  double quote and mixing string

tests: check.sh eliminate useless cat

Useless cat. Consider 'cmd < file | ..' or 'cmd file | ..' instead.

tests: check.sh better if

Prefer [ p ] || [ q ] as [ p -o q ] is not well defined.

tests: check.sh drop unnecessary $

$/${} is unnecessary on arithmetic variables.

tests: check.sh using grep -F

fgrep is non-standard and deprecated. Use grep -F instead.

tests: check.sh no mixing string

Argument mixes string and array. Use * or separate argument.

tests: check.sh assign separately

Declare and assign separately to avoid masking return values.

tests: check.sh double quote

tests: aux.sh fix pid handling

Modification of pids is local (to subshell caused by pipeline).
pids was modified in a subshell. That change might be lost.

tests: aux.sh add -r to read

tests: aux.sh array assing

Assigning an array to a string! Assign as array, or use * instead of @ to concatenate.

tests: aux.sh separately assign

Declare and assign separately to avoid masking return values.

tests: aux.sh using grep -c

Consider using grep -c instead of grep|wc -l.

tests: aux.sh no mixing string

Argument mixes string and array. Use * or separate argument.

tests: aux.sh drop unnecessary $

$/${} is unnecessary on arithmetic variables.

tests: aux.sh use pgrep

tests: aux.sh double quote

clvmd_fix_conf.sh: double quote

clvmd_fix_conf.sh: legacy code

Use $(..) instead of legacy `..`.

vgimportclone.sh: do not quote regex

Don't quote rhs of =~, it'll match literally rather than as a regex.

vgimportclone.sh: use well defined tests

Prefer [ p ] && [ q ] as [ p -a q ] is not well defined.
Prefer [ p ] || [ q ] as [ p -o q ] is not well defined.

vgimportclone.sh: drop extra $

$/${} is unnecessary on arithmetic variables.

vgimportclone.sh: add -r to read

read without -r will mangle backslashes.

vgimportclone.sh: avoid legacy syntax

Use $(..) instead of legacy `..`.

vgimportclone.sh: cannot trap signal 9

vgimportclone.sh: double quote

vgimportclone.sh: no mixing string and array

Argument mixes string and array. Use * or separate argument.

lvm2_cluster_activation_red_hat.sh: double quote

lvmconf_lockingtype2: double quote

lvmconf_lockingtype2: preferable syntax

Prefer [ p ] || [ q ] as [ p -o q ] is not well defined.

lvmdump: replace individual redirects

Consider using { cmd1; cmd2; } >> file instead of individual redirects.

lvmdump: drop useless cat

Useless cat. Consider 'cmd < file | ..' or 'cmd file | ..' instead.

lvmdump: legacy syntax

Use $(..) instead of legacy `..`.

lvmdump: drop extra $

$/${} is unnecessary on arithmetic variables.

lvmdump: replace printf

Don't use variables in the printf format string. Use printf "..%s.." "$foo".
Advice replaced with plain 'echo'.

lvmdump: double quote

Double quote to prevent globbing and word splitting.

lvmconf: prefer well defined and

Prefer [ p ] && [ q ] as [ p -a q ] is not well defined.

lvmconf: double quote

lvmconf: fix test for setting external locking type

Fix test for check of set locking lib dir.

blkdeactivate: quote literal

This { is literal. Check expression (missing ;/\n?) or quote it.

blkdeactivate: more readable if-then-else-fi

blkdeactivate: split local declaration and assign

blkdeactivate: drop unused var

blkdeactivate: double quote

Using double quote to prevent globbing and word splitting.

blkdeactivate: replace obsoleted syntax

fsadm: make if statement more clear

fsadm: add -r to read

read without -r will mangle backslashes

fsadm: no mixing string and array

Argument mixes string and array. Use * or separate argument.

fsadm: split local var dev

Split defining of local variables from being them
assigned at the same time (lossing error code).

fsadm: remove surrounding $()

Remove surrounding $() to avoid executing output

fsadm: fix shell globbing

Fixing reported issue for shellcheck.
Use "" properly to avoid having shell variables split.

post-release

pre-release

command: we don't have all standard macros

So just quite warning

coverity: checked_return of close

Check (or make quiet) close() ret code.
NOTE: there is another duplicated code of daemonize function which
should be converted to libdaemon.

man: regenerate

log: fix message prefix truncation

Restore the 29-character prefix limit that commit
e907535ab977ea29319aeb015cec6c06729f7ce4 ("coverity: use better
function") accidentally reduced to 28 characters.

coverity: model few more functions

Reduce amount of FP.

coverity: move initilization of count variable

Make code more understandble for Coverity, so it sees connection
between 'extents' and 'count' and in fact code is more readable.

coverity: drop unneeded assigns

coverity: validate length of renaming path

Make sure path fits into buffer on stack.

coverity: check malloc results

coverity: use better function

coverity: extra check for find_pool_seg

find_pool_seg may return NULL in some internal error stats.
Handle it explicitely.

debug: more display_lvname usage

coverity: validate retval from get_only_segment_using_this_lv

coverity: add some error path for failed allocs

Coverity reports some unchecked allocations.

debug: use display_lvname

debug: fix error path

Coverity:
log_error may have dereference NULL pointer - use directly
segtype string name.

debug: fail in backup is not traced nor error

debug: tracing error path

debug: check for fail in id validation

cleanup: remove unused code

dev_manager: add missing validation for usable target_name

Coverity shown path where target_name == NULL would be dereferenced.

raid: fix write_commit_backup

With commit 41c10034aa0db3c63f18ff78b64af3a799c154a1 we actually
do require LV to be used with _vg_write_lv_suspend_commit_backup().

So write a proper separte single wrapper for write && commit && backup.

cmdline: fix missing NULL at end of array of args

lvm_run needs to place NULL as the last element into argv[].
Otherwise we get:

Conditional jump or move depends on uninitialised value(s)
 _command_required_pos_matches (lvmcmdline.c:1443)
 _find_command (lvmcmdline.c:1610)
 lvm_run_command (lvmcmdline.c:2770)
 lvm2_run (lvmcmdlib.c:91)

libdm: fix initialization of head for reused structure

Dmeventd reuses  'dm_task' struct for some STATUS operation, but due to
missing reinitization of dm_task target list, it has caused misprocesing
of recieved events as the parsed target has been simply added to the
list of existing status and cause multiple actions being called for
single event.

tests: show more debug info

Helpful for analysis (revert later).

tests: slightly longer delay

Try to avoid race with slightly longer delay.

raid: allow more sync action for extraction

Since we discovered status reporting from 'md' goes from large set
of weird states we can't just decided based on this word.
So let it pass for rebuild and idle as well
and check for health devices afterwards.

tests: correct percent values are reported

tests: unit test for percent printing

cleanup: use display_percent

Replace occurence of %.2f with call of display_percent function.

display: add display_percent function

Add universal function to print dm_percent_t values via single
code path.

TODO: extend with configurale precision of printed values.

libdm: use rounded float for percent print

Use new added  dm_percent_to_round_float to enhance print
of percentage values.

libdm: implement dm_percent_to_round_float

Add function to adjust printing of percent values in better way.
Rounding here is going along following rules:

0% & 100% are always clearly reported with  .0 decimal points.

Values slightly above 0% we make sure a nearest bigger
non zero value with given precission is printed
(i.e. 0.01  for  %.2f  will be shown)

For values closely approaching 100% we again detect and adjust value
that is less then 100 when printed.
(i.e. 99.99  for %.2f will be shown).

For other values we are leaving them with standard rounding mechanism
since we care mainly about corner values 0 & 100 which need to be
printed precisely.

dmeventd: improve more raid status reporting

When we want to report primary leg failure, check for intial 'a',
since otherwice 'Aa idle' is normally visible.

Also reset array of bit flags marking dead devices, once
plugin detects raid is in sync.

tests: use allocate policy in raid test

tests: add transient failure test

Check raid  repairs still 'present', but failed device.

TODO: tests needs more checking about repair actually doing its work.

tests: require also no 'a' for in-sync

Raid easily reports  'idle' with 'a', so in this case
we will not report in-sync.

raid: plugin does not to use --config

Functionality of ignore suspend devices is already granted by:

lvm2_disable_dmeventd_monitoring() -> init_run_by_dmeventd() ->
init_ignore_suspended_devices().

In fact plugins should never use --config because it has
some unpleasant technical issues.