ich habe eine Funktion in meinem Programm welche immer Probleme macht, genauer gesagt, ein static HANDLE macht die Probleme.

Das static HANDLE lässt sich (nachdem ich es als Snapshot benutzt habe für Process32First/Next) nicht beenden mit CloseHandle(), mit GetLastError bekomm ich den Fehlercode 6 -> INVALID_HANDLE_VALUE.

Außerdem müsste ich das Handle wieder auf NULL setzen, nachdem ich es geclosed habe, dabei kommt aber immer ein Programmabsturtz (ist auch klar, wenn das Handle nicht geschlossen wurde)

Weiß vielleicht jemand was ich übersehen hätte können ??

Nachdem letzten Process32Next() aufruf, will ich das Handle schließen und auf NULL setzen. Wie kann es sein das der letzte Funktionsaufruf noch erflogreich ausgeführt wird, und danach ist das Handle invalid ???

Ich hoffe ihr könnt mir helfen, danke schon mal fürs Lesen.

Die Funktion macht im Prinzip nichts anderes als eine eigene struct mit Informationen zu füllen. Die Funktion wird aslo so oft aufgerufen bis es keine Prozesse mehr gibt, dann gibt diese auch false zurück.

Das Handle ist deswegen static, weil es z.B. beim 2. Aufruf nicht wieder mit NULL initialisiert werden soll, es soll ja den Wert behalten, den es von CreateToolhelp32Snapshot() bekommen hat, ansonsten würde ja Process32Next() nicht richtig funktionieren (oder irre ich mich???).

Hier ist der Code, wenn du so schlau bist wie du tust, wirst du den Fehler ja schnell finden...

bool getProc(struct Process* pr) {

    static HANDLE hSnap = NULL;
    PROCESSENTRY32 *proc;
    HANDLE hProcess;
    DWORD ret;
    char *cPath = new char[MAX_PATH];

    proc->dwSize = sizeof(PROCESSENTRY32);

    if (hSnap == NULL) {

        hSnap = CreateToolhelp32Snapshot(TH32CS_SNAPALL,0);

        Process32First(hSnap,proc);

        pr->ProcName = "Leerlaufprozess\0";
        pr->ProcID = proc->th32ProcessID;

        hProcess = OpenProcess(PROCESS_QUERY_INFORMATION,FALSE,proc->th32ProcessID);

        if (hProcess != NULL) {

            ret = GetModuleFileNameEx(hProcess,NULL,cPath,MAX_PATH-1);

        }

        if (ret != 0) {
            pr->ProcPath = cPath;
        } else {
            pr->ProcPath = NULL;
        }

        return true;
    }

    if(Process32Next(hSnap,proc)) {

        pr->ProcName = proc->szExeFile;
        pr->ProcID = proc->th32ProcessID;

        hProcess = OpenProcess(PROCESS_QUERY_INFORMATION | PROCESS_VM_READ ,FALSE,proc->th32ProcessID);

        if (hProcess != NULL) {
            ret = GetModuleFileNameEx(hProcess,NULL,cPath,511);
        }

        if(ret == 0) {
            ret = GetLastError();
            if (ret == 5 || ret == 299) {
                strcpy(cPath,"unable to read path\0");
            } else {
                sprintf(cPath,"ERROR: %i\0",ret);
            }

        }

        pr->ProcPath = cPath;
        return true;

    }

    CloseHandle(hSnap);    // Funktioniert nicht.
    //hSnap = NULL;        
    CloseHandle(hProcess);
    delete proc;
    delete[] cPath;

    return false;

}

PROCESSENTRY32 *proc;

? Der zeigt doch ins Nirgendwo! Und in so einer Situation static zu benutzen suckt einfach und zeugt von extrem schlechtem Stil.

Inf.Student, wieso "suckt" es hier static zu verwenden ? Wann verwendest du static ?

Wenn man sich den Quellcode ansieht, sieht man das ich vergessen habe die HANDLE von hProcess zu schließen, das war der Grund...

bin jetzt kein experte, aber soweit ich weiß sagt static, dass die Variable/Handle auch nach ende der funktion weiter benutzt werden kann
und wenn du dann CloseHandle verwendest machst du ja genau den Vorteil wieder zu nichte.

Vielleicht meckert er deshalb

Nein, es lag daran das ich das andere Handle nicht geschlossen habe.

Mit static deklarierte Variablen werden beim ersten z.B. static int a = 10; initialisiert, wenn ich die Funktion zum 2. mal aufrufe, hat diese zuweisung keinen Einfluss mehr auf die mit static deklarierte Variable.

Das ist auch Absicht, weil diese Funktion ja in einer Schleife so oft aufgerufen wird, bis sie false zurück gibt. Wenn das Handle nicht static wäre, würde es bei jedem Funktionsaufruf wieder mit NULL initialisiert und es kommt zu einer Endlosschleife. Außerdem soll das Handle den Wert so lange beibehalten bis ich es schließe und wieder auf null setzte, wenn ich es nicht mehr benötige.

Ich schließe das Handle, aber das beduetet nicht das ich die Variable hSnap lösche, hSnap bleibt weiterhin ein Handle.