close Warning: Error with navigation contributor "AccountModule"

Opened 8 years ago

#56 new defect

Windows: User can polute restore directory by adding a trailing backslash to the restore destination

Reported by: Achim J Latz Owned by:
Priority: minor Milestone:
Component: bbackupquery Version: trunk
Keywords: windows, restore Cc:

Description

query > ls

00000002 -d---- BACKUPTEST[[BR]]
query > restore BACKUPTEST d:\RESTORE[[BR]]
WARNING: Failed to open file: d:\RESTORE.boxbackupresume: Acceso denegado.(5)[[BR]]
WARNING: Exception thrown: CommonException(OSFileOpenError) at
FileStream.cpp(84)[[BR]]
ERROR:   Failed to save resume info file 'd:\RESTORE.boxbackupresume':[[BR]]
Common OSFileOpenError (Can't  open a file -- attempted to load a
non-existant config file or bad file referenced within?)[[BR]]
ERROR:   Unknown error during restore.[[BR]]


BUT the directory does get created (confirmed on the file system):

query > restore BACKUPTEST d:\RESTORE\[[BR]]
ERROR:   The target directory exists. You cannot restore over an existing directory.[[BR]]


after manually deleting it and adding a trailing slash to the restore destination, it works:

query > restore BACKUPTEST d:\restore\
.......................................................................

With the trailing backslash Box Backup generates d:\RESTORE\.boxbackupresume and without it it tries to generate d:\RESTORE.boxbackupresume?

I have a feeling that this is a workaround that results in the resume file being created in the restored directory, which may not be desirable.

Change History (0)

Note: See TracTickets for help on using tickets.