Gestion des fichiers laissés par le code C++ sous Linux

Question

J'essaie de déboguer un problème dans mon code. J'ai un processus A qui fonctionne continuellement jusqu'à ce que je lui demande d'arrêter.

Inside AI procédez comme suit:

• montage partition/dev/sda1
• open() // crée un fichier vide X
• write() // écrire quelques octets il
• close() // fermer le fichier
• processFile() // Effectuer une opération
• remove() // supprimer le fichier
• umount/dev/sda1

note I test après chaque opération ci-dessus si elle a réussi ou non befor aller de l'avant. Quand je fais lsof | grep A il montre la poignée de fichier de X étant détenue par le processus A. Je vois aussi qu'il a un (supprimé). Cela m'empêche de démonter la partition. Pourquoi cela se passe-t-il et comment puis-je contourner ce problème?

EDIT: Merci à tous. Voici l'extrait du code:

tarFileDesc = _pSysCall->open("test.tar", O_CREAT | O_RDWR | O_APPEND, 0777);  
if (0 > tarFileDesc) 
    return false; 

... some logging here 

// Write http stream to tar file, istr is an argument to my function  
int read_buffer_size = 0;  
buffer = new char[4096];  
while (!istr.eof() && count < content_length) 
{  
    if ((content_length - count) >= 4096) 
     read_buffer_size = 4096; 
    else 
     read_buffer_size = content_length - count; 

    memset(buffer, 0, 4096); 
    istr.read(buffer, read_buffer_size); 
    std::streamsize in_bytes = istr.gcount(); 
    if (istr.fail() || istr.bad()) 
    { 
     status = false; 
     break; 
    } 

    if (write(tarFileDesc, buffer, in_bytes) != in_bytes) 
    { 
     status = false; 
     break; 
    } 
    count += in_bytes; 
} 
// Cleanup buffer 
delete[] buffer; 

if ((0 > tarFileDesc) && (0 != close(tarFileDesc))) 
    return false; 

if (0 != system("tar C /test -xvf test.tar")) 
    return false; 

if (0 != remove("test.tar")) 
     return false; 

Remarque J'ai même essayé de faire juste l'ouvrir, fermer et supprimer. Mais je vois toujours la poignée maintenue par processus.

Answer 1

Vous devrez probablement poster du code. Mais voici une astuce pour vous - si vous créez un fichier temporaire sous Unix/Linux/Mac OS X, vous pouvez le supprimer IMMEDIATEMENT et vous pouvez toujours y écrire, le lire, etc, mais dès que vous le fermez, il sera allez-vous en. Il va également disparaître si une erreur tue votre programme avant que vous ne parveniez à le fermer.

Answer 2

Merci à tous. Voici l'extrait du code:

tarFileDesc = _pSysCall->open("test.tar", O_CREAT | O_RDWR | O_APPEND, 0777);  
if (0 > tarFileDesc) 
    return false; 

... some logging here 

// Write http stream to tar file, istr is an argument to my function  
int read_buffer_size = 0;  
buffer = new char[4096];  
while (!istr.eof() && count < content_length) 
{  
    if ((content_length - count) >= 4096) 
     read_buffer_size = 4096; 
    else 
     read_buffer_size = content_length - count; 

    memset(buffer, 0, 4096); 
    istr.read(buffer, read_buffer_size); 
    std::streamsize in_bytes = istr.gcount(); 
    if (istr.fail() || istr.bad()) 
    { 
     status = false; 
     break; 
    } 

    if (write(tarFileDesc, buffer, in_bytes) != in_bytes) 
    { 
     status = false; 
     break; 
    } 
    count += in_bytes; 
} 
// Cleanup buffer 
delete[] buffer; 

if ((0 > tarFileDesc) && (0 != close(tarFileDesc))) 
    return false; 

if (0 != system("tar C /test -xvf test.tar")) 
    return false; 

if (0 != remove("test.tar")) 
     return false; 

Remarque J'ai même essayé d'ouvrir, de fermer et de retirer. Mais je vois toujours la poignée maintenue par processus.

Answer 3

Le problème se trouve dans ces lignes:

if ((0 > tarFileDesc) && (0 != close(tarFileDesc))) 
    return false; 

Ce ne fermera le descripteur de fichier si tarFileDesc < 0. Ce que vous voulez dire si tarFileDesc >= 0 puis fermez comme il est un descripteur de fichier valide, alors essayez de changer ce code:

if ((0 > tarFileDesc) || (0 != close(tarFileDesc))) 
    return false; 

De cette façon, vous retournez false si le descripteur de fichier est invalide, sinon vous retournez false si vous êtes incapable de le fermer.

Answer 4

Ce regard suspect:

if ((0 > tarFileDesc) && (0 != close(tarFileDesc))) 
    return false; 

regard tourné vers le haut de votre code, vous faites déjà un chèque de 0 > tarFileDesc, et si vrai faux retour juste à ce moment-là. Donc je pense que cette partie du chèque est redondante? c'est à dire., devrait-il simplement lire:

if (0 != close(tarFileDesc))) 
    return false;