| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
A regression introduced in 2.02.89 (11e520256b3005ed813ce83f8770aaab74edef3f)
caused the lvm dumpconfig <node> to print out
the node as well as its subsequent siblings.
The information about "only_one" mode got lost.
Before this patch (just an example node):
# lvm dumpconfig global/use_lvmetad
use_lvmetad=1
thin_check_executable="/usr/sbin/thin_check"
thin_check_options="-q"
(...all nodes to the end of the section)
With this patch applied:
# lvm dumpconfig global/use_lvmetad
use_lvmetad=1
|
| |
|
| |
|
|
|
|
|
|
|
|
| |
pvcreate gives
WARNING: Ignoring unsupported value for metadata/pvmetadataignore.
It was warning if there is no config file entry instead of only if the node
exists but is empty.
|
| |
|
| |
|
|
|
|
| |
Support empty string values.
|
|
|
|
| |
the latter out of libdm.
|
| |
|
|
|
|
| |
(useful for just checking that a given key is of a given type).
|
|
|
|
| |
Since fixed within unreleased version so no WHATS_NEW
|
|
|
|
|
| |
Add function to allow read of empty strings as valid arguments.
Add a warning message if string argument has ignored value.
|
|
|
|
|
|
|
|
| |
Since it's internal function and we always check for NULL value
before call - this is safe.
Just for case add nonnull attribute so analyzer might better
catch error.
|
| |
|
| |
|
|
|
|
|
|
| |
It's 100% equivalent test - since it always happen for the first iteration.
But the check for 'l' is understandable with analyzers - since analyzer
is not smart enough to deduce connection between root->child == NULL.
|
|
|
|
| |
No need to repeat whole declaration for static function.
|
| |
|
| |
|
|
|
|
|
|
| |
Makes dumpconfig whole-section output wrong in a different way from before,
but we should be able to merge cft_cmdline properly into cmd->cft now and
remove cascade.
|
| |
|
|
|
|
| |
Keep the lookup operation const and use const casting at the dm_ function level.
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
functionality. A number of bugs (copied and pasted all over the code) should
disappear:
- most string lookup based on dm_config_find_node would segfault when
encountering a non-zero integer (the intention there was to print an
error message instead)
- check for required sections in metadata would have been satisfied by
values as well (i.e. not sections)
- encountering a section in place of expected flag value would have
segfaulted (due to assumed but unchecked cn->v != NULL)
|
|
|
|
|
| |
dm_config_find_node to give non-const node pointer, since that better reflects
the contract of that function).
|
|
leaving behind the LVM-specific parts of the code (convenience wrappers that
handle `struct device` and `struct cmd_context`, basically). A number of
functions have been renamed (in addition to getting a dm_ prefix) -- namely,
all of the config interface now has a dm_config_ prefix.
|