Zdenek Kabelac
ec02307e95
tests: aux.sh array assing
...
Assigning an array to a string! Assign as array, or use * instead of @ to concatenate.
2017-06-29 22:23:17 +02:00
Zdenek Kabelac
8a912d6dbc
tests: aux.sh separately assign
...
Declare and assign separately to avoid masking return values.
2017-06-29 22:23:17 +02:00
Zdenek Kabelac
4f63b7e9c1
tests: aux.sh using grep -c
...
Consider using grep -c instead of grep|wc -l.
2017-06-29 22:23:17 +02:00
Zdenek Kabelac
b16ca76260
tests: aux.sh no mixing string
...
Argument mixes string and array. Use * or separate argument.
2017-06-29 22:23:17 +02:00
Zdenek Kabelac
44e530b919
tests: aux.sh drop unnecessary $
...
$/${} is unnecessary on arithmetic variables.
2017-06-29 22:23:16 +02:00
Zdenek Kabelac
474e3cd66d
tests: aux.sh use pgrep
2017-06-29 22:23:16 +02:00
Zdenek Kabelac
72a58ce4b0
tests: aux.sh double quote
2017-06-29 22:23:16 +02:00
Zdenek Kabelac
191dadc623
clvmd_fix_conf.sh: double quote
2017-06-29 22:23:16 +02:00
Zdenek Kabelac
99925d0d41
clvmd_fix_conf.sh: legacy code
...
Use $(..) instead of legacy `..`.
2017-06-29 22:23:16 +02:00
Zdenek Kabelac
afa0bd916e
vgimportclone.sh: do not quote regex
...
Don't quote rhs of =~, it'll match literally rather than as a regex.
2017-06-29 22:23:16 +02:00
Zdenek Kabelac
ef4506069a
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.
2017-06-29 22:23:16 +02:00
Zdenek Kabelac
8dc29d7cc7
vgimportclone.sh: drop extra $
...
$/${} is unnecessary on arithmetic variables.
2017-06-29 22:23:16 +02:00
Zdenek Kabelac
16a38dede9
vgimportclone.sh: add -r to read
...
read without -r will mangle backslashes.
2017-06-29 22:23:16 +02:00
Zdenek Kabelac
588b12e71d
vgimportclone.sh: avoid legacy syntax
...
Use $(..) instead of legacy `..`.
2017-06-29 22:23:16 +02:00
Zdenek Kabelac
d745593e89
vgimportclone.sh: cannot trap signal 9
2017-06-29 22:23:16 +02:00
Zdenek Kabelac
275eeb4631
vgimportclone.sh: double quote
2017-06-29 22:23:16 +02:00
Zdenek Kabelac
96a2e00240
vgimportclone.sh: no mixing string and array
...
Argument mixes string and array. Use * or separate argument.
2017-06-29 22:23:16 +02:00
Zdenek Kabelac
4d34bc65dd
lvm2_cluster_activation_red_hat.sh: double quote
2017-06-29 22:23:16 +02:00
Zdenek Kabelac
051c7f88cf
lvmconf_lockingtype2: double quote
2017-06-29 22:23:16 +02:00
Zdenek Kabelac
ef8691a2d2
lvmconf_lockingtype2: preferable syntax
...
Prefer [ p ] || [ q ] as [ p -o q ] is not well defined.
2017-06-29 22:23:16 +02:00
Zdenek Kabelac
030912ec03
lvmdump: replace individual redirects
...
Consider using { cmd1; cmd2; } >> file instead of individual redirects.
2017-06-29 22:23:16 +02:00
Zdenek Kabelac
df57845f61
lvmdump: drop useless cat
...
Useless cat. Consider 'cmd < file | ..' or 'cmd file | ..' instead.
2017-06-29 22:23:16 +02:00
Zdenek Kabelac
a458065aa1
lvmdump: legacy syntax
...
Use $(..) instead of legacy `..`.
2017-06-29 22:23:16 +02:00
Zdenek Kabelac
a0740f831c
lvmdump: drop extra $
...
$/${} is unnecessary on arithmetic variables.
2017-06-29 22:23:16 +02:00
Zdenek Kabelac
e08e714b68
lvmdump: replace printf
...
Don't use variables in the printf format string. Use printf "..%s.." "$foo".
Advice replaced with plain 'echo'.
2017-06-29 22:23:16 +02:00
Zdenek Kabelac
e292c1a1ca
lvmdump: double quote
...
Double quote to prevent globbing and word splitting.
2017-06-29 22:23:16 +02:00
Zdenek Kabelac
6330e2dc86
lvmconf: prefer well defined and
...
Prefer [ p ] && [ q ] as [ p -a q ] is not well defined.
2017-06-29 22:23:16 +02:00
Zdenek Kabelac
ccdb75adcc
lvmconf: double quote
2017-06-29 22:23:16 +02:00
Zdenek Kabelac
de04e36fda
lvmconf: fix test for setting external locking type
...
Fix test for check of set locking lib dir.
2017-06-29 22:23:16 +02:00
Zdenek Kabelac
daa372f78d
blkdeactivate: quote literal
...
This { is literal. Check expression (missing ;/\n?) or quote it.
2017-06-29 22:23:16 +02:00
Zdenek Kabelac
b5fc0d73e5
blkdeactivate: more readable if-then-else-fi
2017-06-29 22:23:16 +02:00
Zdenek Kabelac
dc50f2f4a0
blkdeactivate: split local declaration and assign
2017-06-29 22:23:16 +02:00
Zdenek Kabelac
ce1383fe46
blkdeactivate: drop unused var
2017-06-29 22:23:16 +02:00
Zdenek Kabelac
45861557b5
blkdeactivate: double quote
...
Using double quote to prevent globbing and word splitting.
2017-06-29 22:23:16 +02:00
Zdenek Kabelac
bd359aec86
blkdeactivate: replace obsoleted syntax
2017-06-29 22:23:16 +02:00
Zdenek Kabelac
95ca7cfdff
fsadm: make if statement more clear
2017-06-29 22:23:16 +02:00
Zdenek Kabelac
c6a5964ce9
fsadm: add -r to read
...
read without -r will mangle backslashes
2017-06-29 22:23:16 +02:00
Zdenek Kabelac
afded3e532
fsadm: no mixing string and array
...
Argument mixes string and array. Use * or separate argument.
2017-06-29 22:23:16 +02:00
Zdenek Kabelac
05a6d43476
fsadm: split local var dev
...
Split defining of local variables from being them
assigned at the same time (lossing error code).
2017-06-29 22:23:16 +02:00
Zdenek Kabelac
439aaca3bc
fsadm: remove surrounding $()
...
Remove surrounding $() to avoid executing output
2017-06-29 22:23:16 +02:00
Zdenek Kabelac
b68235c519
fsadm: fix shell globbing
...
Fixing reported issue for shellcheck.
Use "" properly to avoid having shell variables split.
2017-06-29 22:23:15 +02:00
Alasdair G Kergon
0021d707fb
post-release
2017-06-28 17:00:49 +01:00
Alasdair G Kergon
08ce545449
pre-release
2017-06-28 16:38:46 +01:00
Zdenek Kabelac
7ba6896962
command: we don't have all standard macros
...
So just quite warning
2017-06-28 15:05:16 +02:00
Zdenek Kabelac
a533892cd3
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.
2017-06-28 14:42:11 +02:00
Alasdair G Kergon
b90014addc
man: regenerate
2017-06-27 15:47:00 +01:00
Alasdair G Kergon
7f1ec681d6
log: fix message prefix truncation
...
Restore the 29-character prefix limit that commit
e907535ab9
("coverity: use better
function") accidentally reduced to 28 characters.
2017-06-27 12:41:59 +01:00
Zdenek Kabelac
3fe67217d4
coverity: model few more functions
...
Reduce amount of FP.
2017-06-27 12:17:16 +02:00
Zdenek Kabelac
5ff6260071
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.
2017-06-27 12:16:33 +02:00
Zdenek Kabelac
0349b6d889
coverity: drop unneeded assigns
2017-06-27 12:15:42 +02:00