Initialisation d'un tableau de 2 dimensions

Version imprimable

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:

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:

1
2
3
4
5
 
Affichage :
 alalal
edfee
edfee

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

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>
#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:

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

Merci d'avance

18/07/2013, 13h27
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).

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:

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:

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;
}

@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:

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)

18/07/2013, 14h07
ram-0000

OK, ma faute :oops:

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
18/07/2013, 14h12
ram-0000
effectivement, ceci n'est pas correct et conduit à une tentative d'écriture en mémoire RO :
Code:

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:

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

le meiux serait de déclarer le tableau comme un

Code:

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.