[Fedora-directory-commits] ldapserver/ldap/servers/slapd/back-ldbm archive.c, 1.12, 1.13
Noriko Hosoi (nhosoi)
fedora-directory-commits at redhat.com
Wed Oct 3 19:15:00 UTC 2007
Author: nhosoi
Update of /cvs/dirsec/ldapserver/ldap/servers/slapd/back-ldbm
In directory cvs-int.fedora.redhat.com:/tmp/cvs-serv15561
Modified Files:
archive.c
Log Message:
Resolves: #316281
Summary: db2bak fails if the archive path exists and ends with '/'
Fix description:
1. Use path normalize API rel2abspath to remove the trailing '/'s.
2. db2bak renames the archive dir if the directory exists, checks the directory
is the db dir or not. If it is, the command line rename back the existing db
to the original and exits with the error: db2archive: Cannot archive to the db
directory. Then, the original dir is renamed back. If the db2bak runs as a
task (db2bak.pl or console), the server is up and running. Although the
backend is disabled, we don't want to rename the db path even for a short time.
That being said, changed the order to: check if the archive dir is the same as
db dir or not. It exits immediately.
Index: archive.c
===================================================================
RCS file: /cvs/dirsec/ldapserver/ldap/servers/slapd/back-ldbm/archive.c,v
retrieving revision 1.12
retrieving revision 1.13
diff -u -r1.12 -r1.13
--- archive.c 26 Mar 2007 23:04:17 -0000 1.12
+++ archive.c 3 Oct 2007 19:14:58 -0000 1.13
@@ -110,7 +110,7 @@
slapi_task_log_notice(task,
"backup has old idl format; "
"to restore old formated backup onto the new server, "
- "please use command line utility \"bak2db\" .\n");
+ "please use command line utility \"bak2db\" .");
}
goto out;
}
@@ -128,7 +128,7 @@
if (task) {
slapi_task_log_notice(task,
"Backend '%s' is already in the middle of "
- "another task and cannot be disturbed.\n",
+ "another task and cannot be disturbed.",
inst->inst_name);
}
@@ -257,7 +257,8 @@
int ldbm_back_ldbm2archive( Slapi_PBlock *pb )
{
struct ldbminfo *li;
- char *directory = NULL; /* -a <directory> */
+ char *rawdirectory = NULL; /* -a <directory> */
+ char *directory = NULL; /* normalized */
char *dir_bak = NULL;
int return_value = -1;
int task_flags = 0;
@@ -266,25 +267,57 @@
struct stat sbuf;
slapi_pblock_get( pb, SLAPI_PLUGIN_PRIVATE, &li );
- slapi_pblock_get( pb, SLAPI_SEQ_VAL, &directory );
+ slapi_pblock_get( pb, SLAPI_SEQ_VAL, &rawdirectory );
slapi_pblock_get( pb, SLAPI_TASK_FLAGS, &task_flags );
li->li_flags = run_from_cmdline = (task_flags & TASK_RUNNING_FROM_COMMANDLINE);
slapi_pblock_get( pb, SLAPI_BACKEND_TASK, &task );
- if ( !directory || !*directory ) {
+ if ( !rawdirectory || !*rawdirectory ) {
LDAPDebug( LDAP_DEBUG_ANY, "db2archive: no archive name\n",
0, 0, 0 );
- return( -1 );
+ return -1;
}
- if (stat(directory, &sbuf) == 0) {
- int baklen = strlen(directory) + 5; /* ".bak\0" */
+
+ /* start the database code up, do not attempt to perform recovery */
+ if (run_from_cmdline) {
+ /* No ldbm be's exist until we process the config information. */
+ mapping_tree_init();
+ ldbm_config_load_dse_info(li);
+ if (0 != (return_value =
+ dblayer_start(li,
+ DBLAYER_ARCHIVE_MODE|DBLAYER_NO_DBTHREADS_MODE))) {
+ LDAPDebug(LDAP_DEBUG_ANY, "db2archive: Failed to init database\n",
+ 0, 0, 0);
+ if (task) {
+ slapi_task_log_notice(task, "Failed to init database");
+ }
+ return -1;
+ }
+ }
+
+ if (stat(rawdirectory, &sbuf) == 0) {
+ int baklen = 0;
+ directory = rel2abspath(rawdirectory);
+
+ if (slapd_comp_path(directory, li->li_directory) == 0) {
+ LDAPDebug(LDAP_DEBUG_ANY,
+ "db2archive: Cannot archive to the db directory.\n", 0, 0, 0);
+ if (task) {
+ slapi_task_log_notice(task,
+ "Cannot archive to the db directory.");
+ }
+ return_value = -1;
+ goto out;
+ }
+
+ baklen = strlen(directory) + 5; /* ".bak\0" */
dir_bak = slapi_ch_malloc(baklen);
PR_snprintf(dir_bak, baklen, "%s.bak", directory);
LDAPDebug(LDAP_DEBUG_ANY, "db2archive: %s exists. Renaming to %s\n",
directory, dir_bak, 0);
if (task) {
- slapi_task_log_notice(task, "%s exists. Renaming to %s\n",
+ slapi_task_log_notice(task, "%s exists. Renaming to %s",
directory, dir_bak);
}
if (stat(dir_bak, &sbuf) == 0) {
@@ -295,7 +328,7 @@
dir_bak, 0, 0);
if (task) {
slapi_task_log_notice(task,
- "%s exists and failed to delete it.\n", dir_bak);
+ "%s exists and failed to delete it.", dir_bak);
}
return_value = -1;
goto out;
@@ -305,10 +338,10 @@
if (return_value != PR_SUCCESS) {
PRErrorCode prerr = PR_GetError();
LDAPDebug(LDAP_DEBUG_ANY,
- "db2archive: Failed to rename \"%s\" to \"%s\".",
+ "db2archive: Failed to rename \"%s\" to \"%s\".\n",
directory, dir_bak, 0);
LDAPDebug(LDAP_DEBUG_ANY,
- SLAPI_COMPONENT_NAME_NSPR " error %d (%s)",
+ SLAPI_COMPONENT_NAME_NSPR " error %d (%s)\n",
prerr, slapd_pr_strerror(prerr), 0);
if (task) {
slapi_task_log_notice(task,
@@ -336,11 +369,6 @@
goto err;
}
- /* No ldbm be's exist until we process the config information. */
- if (run_from_cmdline) {
- mapping_tree_init();
- ldbm_config_load_dse_info(li);
- }
/* to avoid conflict w/ import, do this check for commandline, as well */
{
Object *inst_obj, *inst_obj2;
@@ -360,7 +388,7 @@
if (task) {
slapi_task_log_notice(task,
"Backend '%s' is already in the middle of "
- "another task and cannot be disturbed.\n",
+ "another task and cannot be disturbed.",
inst->inst_name);
}
@@ -381,45 +409,9 @@
}
}
- /* start the database code up, do not attempt to perform recovery */
- if (run_from_cmdline &&
- 0 != (return_value = dblayer_start(li,DBLAYER_ARCHIVE_MODE|DBLAYER_NO_DBTHREADS_MODE))) {
- LDAPDebug(LDAP_DEBUG_ANY, "db2archive: Failed to init database\n",
- 0, 0, 0);
- if (task) {
- slapi_task_log_notice(task, "Failed to init database");
- }
- goto rel_err;
- }
-
- if (slapd_comp_path(directory, li->li_directory) == 0) {
- LDAPDebug(LDAP_DEBUG_ANY,
- "db2archive: Cannot archive to the db directory.\n", 0, 0, 0);
- if (task) {
- slapi_task_log_notice(task, "Cannot archive to the db directory.\n");
- }
- return_value = -1;
- goto rel_err;
- }
-
/* tell it to archive */
return_value = dblayer_backup(li, directory, task);
-rel_err:
- /* close the database down again */
- if (run_from_cmdline &&
- 0 != dblayer_close(li,DBLAYER_ARCHIVE_MODE|DBLAYER_NO_DBTHREADS_MODE)) {
- LDAPDebug(LDAP_DEBUG_ANY, "db2archive: Failed to close database\n",
- 0, 0, 0);
- if (task) {
- slapi_task_log_notice(task, "Failed to close database");
- }
-
- /* The backup succeeded, so a failed close is not really a
- total error... */
- /*return( -1 );*/
- }
-
if (! run_from_cmdline) {
ldbm_instance *inst;
Object *inst_obj;
@@ -436,13 +428,24 @@
LDAPDebug(LDAP_DEBUG_ANY, "db2archive: Rename %s back to %s\n",
dir_bak, directory, 0);
if (task) {
- slapi_task_log_notice(task, "Rename %s back to %s\n",
+ slapi_task_log_notice(task, "Rename %s back to %s",
dir_bak, directory);
}
ldbm_delete_dirs(directory);
PR_Rename(dir_bak, directory);
}
out:
+ /* close the database down again */
+ if (run_from_cmdline &&
+ 0 != dblayer_close(li,DBLAYER_ARCHIVE_MODE|DBLAYER_NO_DBTHREADS_MODE)) {
+ LDAPDebug(LDAP_DEBUG_ANY, "db2archive: Failed to close database\n",
+ 0, 0, 0);
+ if (task) {
+ slapi_task_log_notice(task, "Failed to close database");
+ }
+ }
+
slapi_ch_free_string(&dir_bak);
+ slapi_ch_free_string(&directory);
return return_value;
}
More information about the Fedora-directory-commits
mailing list