DeleteFile falla en el archivo cerrado recientemente

Question

Tengo un único programa de subproceso (C++, Win32, NTFS) que primero crea un archivo temporal bastante largo, lo cierra, abre para leer, lee, cierra de nuevo e intenta eliminar utilizando DeleteFile().

Suele pasar sin problemas, pero a veces DeleteFile() falla, y GetLastError() devuelve ERROR_ACCESS_DENIED. El archivo no es de solo lectura seguro. Ocurre en archivos de cualquier tamaño, pero la probabilidad aumenta con el tamaño del archivo.

¿Alguna idea de lo que puede estar bloqueando el archivo? Intenté con las herramientas de WinInternals para verificar y no encontré nada sospechoso.

Answer 1

¿Quizás los cambios todavía estén en caché y aún no se hayan guardado?

Puede verificar esto agregando un WaitForSingleObject en el controlador del archivo para estar seguro.

Answer 2

Agregue una llamada a MessageBox() antes de invocar DeleteFile(), cuando aparezca, ejecute la herramienta sysinternals Process Explorer. Busque un identificador abierto para el archivo. Con toda probabilidad, no ha cerrado todos los identificadores del archivo ...

Answer 3

Simplemente una conjetura: ¿tiene algún software antivirus instalado? ¿Has intentado desactivar las características de protección en tiempo real, si lo haces?

Answer 4

Windows es notorio por este problema. sqlite maneja el problema volviendo a intentar la operación de eliminación cada 100 milisegundos hasta un número máximo.

Creo que si está seguro de que no tiene controladores abiertos, hacer esto en su implementación le ahorrará algunos dolores de cabeza cuando cosas como el software antivirus abren el archivo.

Como referencia, el comentario de fuente sqlite:

/*                  
** Delete the named file.            
**                  
** Note that windows does not allow a file to be deleted if some other 
** process has it open. Sometimes a virus scanner or indexing program 
** will open a journal file shortly after it is created in order to do 
** whatever it does. While this other process is holding the   
** file open, we will be unable to delete it. To work around this  
** problem, we delay 100 milliseconds and try to delete again. Up  
** to MX_DELETION_ATTEMPTs deletion attempts are run before giving  
** up and returning an error.           
*/ 

Answer 5

Creo que esto está cubierto en Windows Internals. La historia corta es que, aunque haya llamado a CloseHandle en el manejador de archivos, el kernel puede tener referencias pendientes que tarden unos pocos milisegundos en cerrarse.

Una manera más confiable de eliminar el archivo cuando haya terminado es utilizar el indicador FILE_FLAG_DELETE_ON_CLOSE al abrir el último identificador. Esto funciona incluso mejor si puede evitar cerrar el archivo entre lecturas/escrituras.

#include <windows.h> 
#include <stdio.h> 

int wmain(int argc, wchar_t** argv) 
{ 
    LPCWSTR fileName = L"c:\\temp\\test1234.bin"; 

    HANDLE h1 = CreateFileW(
     fileName, 
     GENERIC_WRITE, 
     // make sure the next call to CreateFile can succeed if this handle hasn't been closed yet 
     FILE_SHARE_READ | FILE_SHARE_DELETE, 
     NULL, 
     CREATE_ALWAYS, 
     FILE_FLAG_SEQUENTIAL_SCAN | FILE_ATTRIBUTE_TEMPORARY, 
     NULL); 
    if (h1 == INVALID_HANDLE_VALUE) 
    { 
     fprintf(stderr, "h1 failed: 0x%x\n", GetLastError()); 
     return GetLastError(); 
    } 

    HANDLE h2 = CreateFileW(
     fileName, 
     GENERIC_READ, 
     // FILE_SHARE_WRITE is required in case h1 with GENERIC_WRITE access is still open 
     FILE_SHARE_WRITE | FILE_SHARE_READ | FILE_SHARE_DELETE, 
     NULL, 
     OPEN_EXISTING, 
     // tell the OS to delete the file as soon as it is closed, no DeleteFile call needed 
     FILE_FLAG_DELETE_ON_CLOSE | FILE_FLAG_SEQUENTIAL_SCAN | FILE_ATTRIBUTE_TEMPORARY, 
     NULL); 
    if (h2 == INVALID_HANDLE_VALUE) 
    { 
     fprintf(stderr, "h2 failed: 0x%x\n", GetLastError()); 
     return GetLastError(); 
    } 

    return 0; 
} 

Answer 6

#include <iostream> 
#include <windows.h> 

int main(int argc, const char * argv[]) 
{ 
    // Get a pointer to the file name/path 
    const char * pFileToDelete = "h:\\myfile.txt"; 
    bool RemoveDirectory("h:\\myfile.txt"); 

    // try deleting it using DeleteFile 
    if(DeleteFile(pFileToDelete)) 
    { 
     // succeeded 
     std::cout << "Deleted file" << std::endl; 
    } 
    else 
    { 
     // failed 
     std::cout << "Failed to delete the file" << std::endl; 
    } 
    std::cin.get(); 
    return 0; 
}