Bus error et strcmp

Version imprimable

Bus error et strcmp

Bonjour,

j'ai un petit soucis avec la fonction strcmp qui me lance un "Bus error"
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 #include <stdio.h> #include <string.h> int main(void) { puts("Running"); char* s; while(1) { puts("mydevice> "); fgets(s, 10, stdin); if ( strcmp(s,"IDN")==0 ) { puts("*IDN? identification \r\n"); } else { puts("Error ! this firmware doesn't understand this command\r\n"); } } puts("Stopping\r\n"); /* it should never happen */ return 0; }
Avez-vous une idée d'où peut venir mon problème (ça fait un petit moment que je tourne autour sans le voir... ?

tant que j'y suis
je compile avec gcc avec les flags -ansi -pedantic -Wall
et j'ai le warning suivant à la compilation :
warning: ISO C90 forbids mixed declarations and code
c'est à cause de char* s; mais je ne vois pas en quoi je mélange
code et déclaration...

... pour cette deuxième partie j'ai vu... il suffit que je place ma déclaration avant la fonction main

Merci d'avance

18/04/2009, 14h45
diogene
Code:

1 2 3 char* s; .... fgets(s, 10, stdin);
s doit être un tableau de caractères, pas un simple pointeur.
Code:

1 2 3 char s[10]; .... fgets(s, 10, stdin);
Citation:

c'est à cause de char* s; mais je ne vois pas en quoi je mélange

En C90, les définitions des variables doivent être faites en début de bloc, avant la première instruction du bloc. Ici, char *s est précédé de l'appel à puts

Merci j'avais pensé à ça

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
 
#include <stdio.h> 
#include <string.h>
 
#define STR_SIZE 10
char s[STR_SIZE];
 
int main(void)
{
  puts("Running");
 
  while(1) {
    puts("mydevice> ");
    fgets(s, STR_SIZE, stdin);
 
    if ( strcmp(s,"IDN")==0 ) {
      puts("*IDN? identification \r\n");
    } else {
      puts("Error ! this firmware doesn't understand this command\r\n");
    }
  }
 
  puts("Stopping\r\n"); /* it should never happen */
 
  return 0;
}

je n'ai effectivement plus le bus error par contre je ne valide jamais la condition !!!

Une idée ?

18/04/2009, 15h03
Melem

fgets lit aussi le '\n', c'est-à-dire que tu as dans s "IDN\n" et non "IDN". Il y a plusieurs entrées qui traitent de la bonne utilisation de fgets dans la FAQ ...

Merci beaucoup

Voici le code final, j'ai préféré éviter les puts et fgets....

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 <stdio.h> 
#include <string.h>
#include <ctype.h>
 
#define STR_SIZE 10
char s[STR_SIZE]; /* received string */
char s2[STR_SIZE]; /* string converted to upper case */
int i;
 
int main(void)
{
  printf("Running\n");
 
  while(1) {
    printf("mydevice> ");
    scanf("%s",s);
    for (i=0 ; i<STR_SIZE ; ++i)
    {
      s2[i]=toupper(s[i]);
    }
    if ( strcmp(s2, "*IDN?")==0 ) {
      printf("*IDN? = device identification\n");
    } else {
      fprintf(stderr,"Error ! this firmware doesn't understand this command\n");
    }
  }
 
  printf("Stopping\n"); /* it should never happen */
 
  return 0;
}