Fixed #8588: Error doing nbackup when Database file name has extended ASCII chars

Author: AlexPeshkoff

src/utilities/nbackup/nbackup.cpp

@@ -296,6 +296,8 @@ class NBackup
 				pr_error(status, "nbackup needs local access to database file");
 		}
 
+		toSystem(decompress);
+		toSystem(db);
 		expandDatabaseName(db, dbname, NULL);
 
 		if (!uSvc->isService())
@@ -364,7 +366,7 @@ class NBackup
 	void internal_unlock_database();
 	void attach_database();
 	void detach_database();
-	string to_system(const PathName& from);
+	void toSystem(AbstractString& from);
 
 	// Create/open database and backup
 	void open_database_write();
@@ -580,12 +582,10 @@ void NBackup::close_database()
 #endif
 }
 
-string NBackup::to_system(const PathName& from)
+void NBackup::toSystem(AbstractString& from)
 {
-	string to = from.ToString();
 	if (uSvc->utf8FileNames())
-		ISC_utf8ToSystem(to);
-	return to;
+		ISC_utf8ToSystem(from);
 }
 
 
@@ -597,14 +597,13 @@ void NBackup::open_backup_scan()
 		return;
 	}
 
-	string nm = to_system(bakname);
 #ifdef WIN_NT
-	backup = CreateFile(nm.c_str(), GENERIC_READ, 0,
+	backup = CreateFile(bakname.c_str(), GENERIC_READ, 0,
 		NULL, OPEN_EXISTING, FILE_ATTRIBUTE_NORMAL | FILE_FLAG_SEQUENTIAL_SCAN, NULL);
 	if (backup != INVALID_HANDLE_VALUE)
 		return;
 #else
-	backup = os_utils::open(nm.c_str(), O_RDONLY | O_LARGEFILE);
+	backup = os_utils::open(bakname.c_str(), O_RDONLY | O_LARGEFILE);
 	if (backup >= 0)
 		return;
 #endif
@@ -752,7 +751,6 @@ void NBackup::open_backup_decompress()
 
 void NBackup::create_backup()
 {
-	string nm = to_system(bakname);
 #ifdef WIN_NT
 	if (bakname == "stdout") {
 		backup = GetStdHandle(STD_OUTPUT_HANDLE);
@@ -767,7 +765,7 @@ void NBackup::create_backup()
 		// avoids an issue where writing to a file across a network can occasionally 
 		// return ERROR_ACCESS_DENIED.
 
-		backup = CreateFile(nm.c_str(), GENERIC_READ | GENERIC_WRITE, FILE_SHARE_DELETE,
+		backup = CreateFile(bakname.c_str(), GENERIC_READ | GENERIC_WRITE, FILE_SHARE_DELETE,
 			NULL, CREATE_NEW, FILE_ATTRIBUTE_NORMAL | FILE_FLAG_SEQUENTIAL_SCAN, NULL);
 	}
 	if (backup != INVALID_HANDLE_VALUE)
@@ -778,7 +776,7 @@ void NBackup::create_backup()
 		backup = 1; // Posix file handle for stdout
 		return;
 	}
-	backup = os_utils::open(nm.c_str(), O_WRONLY | O_CREAT | O_EXCL | O_LARGEFILE, 0660);
+	backup = os_utils::open(bakname.c_str(), O_WRONLY | O_CREAT | O_EXCL | O_LARGEFILE, 0660);
 	if (backup >= 0)
 		return;
 #endif
@@ -934,6 +932,7 @@ void NBackup::attach_database()
 	}
 
 	ClumpletWriter dpb(ClumpletReader::dpbList, MAX_DPB_SIZE);
+	uSvc->fillDpb(dpb);
 
 	const unsigned char* authBlock;
 	unsigned int authBlockSize = uSvc->getAuthBlock(&authBlock);
@@ -1181,7 +1180,10 @@ void NBackup::backup_database(int level, const PathName& fname)
 		detach_database();
 
 		if (fname.hasData())
+		{
 			bakname = fname;
+			toSystem(bakname);
+		}
 		else
 		{
 			// Let's generate nice new filename
@@ -1557,6 +1559,8 @@ void NBackup::restore_database(const BackupFiles& files)
 						delete[] page_buffer;
 						return;
 					}
+					toSystem(bakname);
+
 					// Never reaches this point when run as service
 					try {
 						fb_assert(!uSvc->isService());
@@ -1591,6 +1595,7 @@ void NBackup::restore_database(const BackupFiles& files)
 					return;
 				}
 				bakname = files[curLevel];
+				toSystem(bakname);
 				open_backup_scan();
 			}