C++ GetProcessMemoryInfo échoue

Version imprimable

Bonjour,

je cherche à obtenir des informations sur l'utilisation mémoire de mon processus :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
#include <iostream>

using namespace std;

#include <windows.h>
#include <psapi.h> // For PPROCESS_MEMORY_COUNTERS

int main()
{
    // Variables
    DWORD processId; // Id of the process
    HANDLE hProcess; // Handle to the process
    PPROCESS_MEMORY_COUNTERS ppsmemCounters; // Pointer to a PROCESS_MEMORY_COUNTERS structure

    // Get and display the current process Id
    processId=GetCurrentProcessId();
    cout << "Current process Id : " << processId << endl;

    // Open the process
    if( (hProcess=OpenProcess(PROCESS_QUERY_INFORMATION, TRUE, processId)) == NULL){
        cout << "Unable to open the process : " << GetLastError() << endl;
    }

    // Get information about the memory usage of this process
    if(GetProcessMemoryInfo(hProcess, ppsmemCounters, sizeof(ppsmemCounters))==0){
        cout << "Unable to get the process memory info : " << GetLastError() << endl;
    }


    return 0;
}

Résultat : "Unable to get the process memory info : 998".

Pouquoi?

Merci.

31/05/2009, 09h29
nicolas.sitbon

Peut être que cela pourra t'aider :

Citation:

Windows Server 2003 and Windows XP/2000: The handle must have the PROCESS_QUERY_INFORMATION and PROCESS_VM_READ access rights.
31/05/2009, 12h38
Médinoc

Pourquoi ne pas utiliser directement GetCurrentProcess() plutôt que réouvrir un nouveau handle vers le processus?
31/05/2009, 21h39
YuGiOhJCJ

Oh effectivement je n'ai que "PROCESS_QUERY_INFORMATION" il me manque donc peut être "PROCESS_VM_READ". Je vais essayer d'ajouter l'autre.

Heu oui je pense que GetCurrentProcess sera mieux. Je vais essayer ça.

Je n'ai plus le même message d'erreur ajourd'hui :

Code:

Unable to get the process memory info : 122

Cette fois ce n'est plus un :
Code:

1 2 ERROR_NOACCESS 998 (0x3E6) Invalid access to memory location.
c'est un :
Code:

1 2 ERROR_INSUFFICIENT_BUFFER 122 (0x7A) The data area passed to a system call is too small.
Je ne sais pas si j'ai bien fait pour ajouter deux droits d'accès :
Code:

1 2 3 4 5 // Open the process if( (hProcess=OpenProcess(PROCESS_QUERY_INFORMATION + PROCESS_VM_READ , TRUE, processId)) == NULL){ cout << "Unable to open the process : " << GetLastError() << endl; }
Est-ce que c'est ainsi la syntaxe lorsque je dois préciser plusieurs droits d'accès?
Pourquoi le code d'erreur a subitement changé à votre avis?
Il semblerait que je dois augmenter la taille des données passées au système...Comment faire cela?

Merci.

01/06/2009, 12h00
Médinoc

Le code d'erreur a changé parce que tu n'as plus d'erreur d'accès (même si à ta place, j'utiliserais '|' (le OU binaire bit-à-bit) pour combiner les flags, plutôt que '+'.

Tu as une erreur parce que ton sizeof est incorrect.
01/06/2009, 12h59
YuGiOhJCJ

Ok j'utilise le caractère "|" (pipe) maintenant.
En lisant l'exemple suivant :
http://msdn.microsoft.com/en-us/libr...50(VS.85).aspx
Je remarque pourtant que le sizeof() s'applique bien sur le pointeur vers une structure PROCESS_MEMORY_COUNTERS. C'est bien ce que je fais donc en théorie je ne devrai pas avoir l'erreur 122.
Comment expliquer cela?
01/06/2009, 13h40
nicolas.sitbon

Citation:

Envoyé par YuGiOhJCJ

Ok j'utilise le caractère "|" (pipe) maintenant.
En lisant l'exemple suivant :
http://msdn.microsoft.com/en-us/libr...50(VS.85).aspx
Je remarque pourtant que le sizeof() s'applique bien sur le pointeur vers une structure PROCESS_MEMORY_COUNTERS. C'est bien ce que je fais donc en théorie je ne devrai pas avoir l'erreur 122.
Comment expliquer cela?

dans l'exemple que tu donnes, ils donnent l'adresse de PPROCESS_MEMORY_COUNTERS, toi tu le passes directement.

Effectivement. C'est corrigé merci :)

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
int main()
{
    // Variables
    DWORD processId; // Id of the process
    HANDLE hProcess; // Handle to the process
    PROCESS_MEMORY_COUNTERS psmemCounters;

    // Get and display the current process Id
    processId=GetCurrentProcessId();
    cout << "Current process Id : " << processId << endl;

    // Open the process
    if( (hProcess=OpenProcess(PROCESS_QUERY_INFORMATION|PROCESS_VM_READ , TRUE, processId)) == NULL){
        cout << "Unable to open the process : " << GetLastError() << endl;
    }

    // Get information about the memory usage of this process
    if(GetProcessMemoryInfo(hProcess, &psmemCounters, sizeof(psmemCounters))==0){
        cout << "Unable to get the process memory info : " << GetLastError() << endl;
    }

    return 0;
}

Plus d'erreur retournée.
Résolu.