diff options
| author | Dave Wysochanski <dwysocha@redhat.com> | 2008-12-11 03:32:56 +0000 |
|---|---|---|
| committer | Dave Wysochanski <dwysocha@redhat.com> | 2008-12-11 03:32:56 +0000 |
| commit | 4fa2c7f8b663b8ab3e0794acdc3ee8e08bdcfbc5 (patch) | |
| tree | 59596c7445de4e0f209aa9dfbde538db58f16e91 /daemons/clvmd/lvm-functions.c | |
| parent | 058f3d3523862a4f871d94289c1d5c39584fdff9 (diff) | |
| download | lvm2-4fa2c7f8b663b8ab3e0794acdc3ee8e08bdcfbc5.tar.gz lvm2-4fa2c7f8b663b8ab3e0794acdc3ee8e08bdcfbc5.tar.xz lvm2-4fa2c7f8b663b8ab3e0794acdc3ee8e08bdcfbc5.zip | |
Remove archive_enable() calls after create_toolcontext() calls.
_init_backup() calls archive_init(), which originally set 'enabled' to
a hardcoded '1' value. This seems incorrect based on my read of other
areas of the code so here we add a 'enabled' paramter to archive_init().
We pass in cmd->default_settings.archive, which is obtained from the
config tree. Later in create_toolcontext, cmd->current_settings is
set to cmd->default_settings. The archive_enable() call we remove
here was using cmd->current_settings to set the 'archive' enable
value. The final value of cmd->archive_params->enabled should thus
be equivalent to the original code.
Diffstat (limited to 'daemons/clvmd/lvm-functions.c')
| -rw-r--r-- | daemons/clvmd/lvm-functions.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/daemons/clvmd/lvm-functions.c b/daemons/clvmd/lvm-functions.c index 9da99dd7..ef4953b3 100644 --- a/daemons/clvmd/lvm-functions.c +++ b/daemons/clvmd/lvm-functions.c @@ -733,7 +733,6 @@ int init_lvm(int using_gulm) init_syslog(LOG_DAEMON); openlog("clvmd", LOG_PID, LOG_DAEMON); set_activation(cmd->current_settings.activation); - archive_enable(cmd, cmd->current_settings.archive); backup_enable(cmd, cmd->current_settings.backup); cmd->cmd_line = (char *)"clvmd"; |