Discussion :

[zandoli]

Bonjour,
J'ai un prob avec la fonction CreateFile() qui me sort la même erreur quand j'essaie d'ouvrir un port série:

A chaque fois il m'affiche l'erreur numéro 5 du GetLastError() qui est ERROR_ACCESS_DENIED (msdn).

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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
33
HANDLE com;

// MAIN:
do{
        printf("Choisissez le port de communication ('COM1' ou 'COM2'):\n");
        gets(id_port);
        erreur = initcom(id_port);  
        CloseHandle(com);  
}while (erreur==1);

// FONCTION INITCOM:

// Ouverture du port
     com = CreateFile( id_port,                      // Nom du port                                          
                      GENERIC_READ | GENERIC_WRITE, // Mode d'accès
                      0,
                      0,
                      OPEN_EXISTING,                // Ouvre un fichier existant
                      FILE_ATTRIBUTE_NORMAL,
                      0);
     
     // Test erreur d'ouverture
     if(com == INVALID_HANDLE_VALUE){
            if(GetLastError() == ERROR_FILE_NOT_FOUND){
                   printf("\n\nErreur! Le port %s n'existe pas.\n\n", id_port); 
                   return(1);
            }
     
            printf("\n\nErreur! Probleme d'acces au port %s.\n"
                                "Erreur: %d.\n\n", id_port, GetLastError());
            return(1);
     }

[Foobar1329]

Hello,

Bonjour,
J'ai un prob avec la fonction CreateFile() qui me sort la même erreur quand j'essaie d'ouvrir un port série:

A chaque fois il m'affiche l'erreur numéro 5 du GetLastError() qui est ERROR_ACCESS_DENIED (msdn).

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
HANDLE com;

// MAIN:
do{
        printf("Choisissez le port de communication ('COM1' ou 'COM2'):\n");
        gets(id_port);
        erreur = initcom(id_port);  
        CloseHandle(com);  
}while (erreur==1);

Ne jamais utiliser gets(), fonction INTERDITE. Tu risqus de faire un beau débordement de tampon car tu n'as aucun contrôle sur la saisie. Utiliser fgets() sur le flux stdin plutôt. Sans compter qu'ici on ne sait pas ce qu'est id_port.

Par ailleurs, ce n'est pas très clair : tu fais un CloseHandle() sur com, alors qu'on ne le voit initialisé nulle part, même si on se doute après que c'est la fonction initcom() qui s'en charge. Pareil pour le do/while, peu lisible, il faut préférer faire une première tentative et travailler avec le while, plus clair à mon goût et moins piégeux.

Exemple de code :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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
HANDLE hCom = NULL;
char id_port[80+1] = { '\0' };

printf("Choisissez le port de communication ('COM1' ou 'COM2'): ");
fflush(stdout);
if ( fgets(id_port, sizeof(id_port)/sizeof(*id_port), stdin) )
{
    if (strncmp(id_port,"COM1",4) == 0 ||  strncmp(id_port,"COM2",4) == 0)
    {
         hCom = CreateFile(id_port,
                                   GENERIC_READ | GENERIC_WRITE,
                                   0,
                                   NULL,
                                   OPEN_EXISTING,
                                   0,
                                   NULL);
        
        if (hCom == INVALID_HANDLE_VALUE) 
        {
            /* gestion erreur*/
        }   
        else
        {
             /* continuer */
             /* ... */ 
             CloseHandle(hCom); hCom = NULL; 
        }  
}

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
// FONCTION INITCOM:

// Ouverture du port
     com = CreateFile( id_port,                      // Nom du port                                          
                      GENERIC_READ | GENERIC_WRITE, // Mode d'accès
                      0,
                      0,
                      OPEN_EXISTING,                // Ouvre un fichier existant
                      FILE_ATTRIBUTE_NORMAL,
                      0);
     
     // Test erreur d'ouverture
     if(com == INVALID_HANDLE_VALUE){
            if(GetLastError() == ERROR_FILE_NOT_FOUND){
                   printf("\n\nErreur! Le port %s n'existe pas.\n\n", id_port); 
                   return(1);
            }
     
            printf("\n\nErreur! Probleme d'acces au port %s.\n"
                                "Erreur: %d.\n\n", id_port, GetLastError());
            return(1);
     }

Je ne sais pas si CreateFile et bien utilisé, bien que ça en ait l'air. Il faut regarder les exemples sur MSDN : http://windowssdk.msdn.microsoft.com.../ms680815.aspx

CreateFile :
http://windowssdk.msdn.microsoft.com.../ms685006.aspx

Essaie d'ouvrir le port en en lecture seule ou en écriture seule pour voir.

A+
Foobar1329