Discussion :

[Alaa-linux]

Bonjour a tous ,

j'ai rencontré un probléme lors de l'initialisation d'un tableau de 2 dimensions , mon code je le compile sous linux avec le compilateur GCC .

je poste ci-dessous les deux programme le premier marche é le deuxiéme non .

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
 #include <stdio.h>
#include <string.h>
#include <stdint.h>
#include <stdio.h>
 
#define NB_LIGNES 2048
#define NB_COLS   2048
 
int main(void)
{
  char* cases[2048][30]; 
     cases[0][0]="alalal";
     cases[1][0]="edfee";
     cases[2][0]="edfee";
 
   printf("%s\n",cases[0][0]);
   printf("%s\n",cases[1][0]);
   printf("%s\n",cases[2][0]);
 
  return 0;
}

Code :

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

1
2
3
4
5
 
Affichage :
 alalal
edfee
edfee

Programmme 2 avec utilisation de strcpy ne marche pas !!!!!

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
 
#include <stdio.h>
#include <string.h>
#include <stdint.h>
#include <stdio.h>
 
#define NB_LIGNES 2048
#define NB_COLS   2048
 
int main(void)
{
  char* cases[2048][30]; 
 
 
 
     cases[0][0]="alalal";
     cases[1][0]="edfee";
     cases[2][0]="edfee";
 
    strcpy(cases[0][0],"alalal");
    strcpy(cases[1][0],"blblbl");
     strcpy(cases[2][0],"dlsmsm");
 
 
  return 0;
}

Code :

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

1
2
3
 
Affichage sortie :
Segmentation fault (core dumped)

Merci d'avance

[supersnail]

Bonjour,

Ce comportement est tout à fait normal, étant donné que tes chaînes de caractères sont dans une section uniquement accessible en lecture seule (.rodata généralement).

Ainsi toute tentative d'écriture dans cette zone mémoire (ce que tu fais avec un strcpy) échouera lamentablement (sans parler du fait que tu recopies des chaînes plus grandes que la taille du buffer, ce qui irait joyeusement écraser les données derrière si la zone mémoire était accessible en écriture).

[Alaa-linux]

Merci pour vos réponses ,

@ ram-0000 c'est pas un probleme de '&' avant les strcpy , maintenant avec la '&' le programme ne se compile même pas : voici l'erreur :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
main.c:11:6: warning: passing argument 1 of ‘strcpy’ from incompatible pointer type [enabled by default]
In file included from main.c:2:0:
/usr/include/string.h:128:14: note: expected ‘char * __restrict__’ but argument is of type ‘char **’
main.c:12:6: warning: passing argument 1 of ‘strcpy’ from incompatible pointer type [enabled by default]
In file included from main.c:2:0:
/usr/include/string.h:128:14: note: expected ‘char * __restrict__’ but argument is of type ‘char **’
main.c:13:6: warning: passing argument 1 of ‘strcpy’ from incompatible pointer type [enabled by default]
In file included from main.c:2:0:
/usr/include/string.h:128:14: note: expected ‘char * __restrict__’ but argument is of type ‘char **’

le programme :

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
 #include <stdio.h>
#include <string.h>
 
 
int main(void)
{
  char* cases[2048][30]; 
 
     strcpy(&cases[0][0],"alalal");
     strcpy(&cases[1][0],"blblbl");
     strcpy(&cases[2][0],"dlsmsm");
 
 
   printf("%s\n",cases[0][0]);
   printf("%s\n",cases[1][0]);
   printf("%s\n",cases[2][0]);
 
 
  return 0;
}

[supersnail]

@ram-000: la variable cases est un tableau à 2 dimensions de char* donc ça vient pas de là.

Et pour info, j'ai regardé avec gdb ce qui se passait:

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
Program received signal SIGSEGV, Segmentation fault.
--------------------------------------------------------------------------[ regs]
  EAX: 0x080484D0  EBX: 0xB7FAE000  ECX: 0x89AB2199  EDX: 0xBFFFF3D4  o d I t S z a p c 
  ESI: 0x00000000  EDI: 0x00000000  EBP: 0xBFFFF3A8  ESP: 0xBFFC33A8  EIP: 0x080483FD
  CS: 0073  DS: 007B  ES: 007B  FS: 0000  GS: 0033  SS: 007B
--------------------------------------------------------------------------[ code]
=> 0x80483fd <main+45>:	mov    DWORD PTR [eax],0x6c616c61
   0x8048403 <main+51>:	mov    WORD PTR [eax+0x4],0x6c61
   0x8048409 <main+57>:	mov    BYTE PTR [eax+0x6],0x0
   0x804840d <main+61>:	mov    eax,DWORD PTR [ebp-0x3bf88]
   0x8048413 <main+67>:	mov    DWORD PTR [eax],0x6c626c62
   0x8048419 <main+73>:	mov    WORD PTR [eax+0x4],0x6c62
   0x804841f <main+79>:	mov    BYTE PTR [eax+0x6],0x0
   0x8048423 <main+83>:	mov    eax,DWORD PTR [ebp-0x3bf10]
--------------------------------------------------------------------------------
0x080483fd in main ()
gdb$ x/s $eax
0x80484d0:	"alalal"

Donc c'est bien un problème de "read only" qui intervient ici (et on peut là-aussi inspecter le "program header" du programme)

[ram-0000]

OK, ma faute

J'avais pas vu que c'était un tableau de "char *", je n'avais vu que le char.

La syntaxe strcpy telle que tu l'avais écrite est donc bonne

Par contre, il faut allouer de la mémoire pour chaque case du tableau avant de faire la copie par strcpy

[ram-0000]

effectivement, ceci n'est pas correct et conduit à une tentative d'écriture en mémoire RO :

Code :

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

1
2
cases[0][0]="alalal";
strcpy(cases[0][0],"alalal");

La première instruction stocke dans cases[0][0] un pointeur sur une chaine de caractère read only

La deuxième instruction copie une chaine de caractères dans le pointeur contenu par cases[0][0] (donc un pointeur sur une zone read only)

Si tu fais comme cela :

Code :

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

cases[0][0]="alalal";

le meiux serait de déclarer le tableau comme un

Code :

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

const char *cases[2048][30]

, comme ceci, toute tentative de strcpy sera détectée par le compilateur.

Si tu veux absoluiment faire par strcpy, il faut faire un malloc avant le strcpy et ne pas oublier le free à la fin.