Discussion :

[SebastienLeto]

Bonjour,

J'obtiens cette erreur de mémoire dans la partie suivante de mon programme :
J'ai une matrice Q avec un nombre de ligne :Code->M (=252)*GF->M (=8) et un nombre de colonne : nbColumn (=5).
J'ai une matrice H de taille Code->M (=252) par nbColumn (=5).
J'ai une matrice Decoder->LR_INIT de taille GF->M (=8) par Code->N (=512).

Je remplis la matrice Q avec les valeurs contenues dans LR_INIT. Mais comme elles ne sont pas de même taille au niveau des colonnes, LR_INIT ayant plus de colonnes, je séléctionne seulement certains éléments de LR_INIT. Le numéro de colonne de l'élément sélectionné est contenu dans la matrice qui contient les indices des colonnes à séléctionnner, c'est à dire H.

J'obtiens l'erreur indiqué pour k=1, i=4, j=4, qui correspond à une valeur nulle dans H[k][j]. Le programme me dit alors qu'il ne peut évaluer Q[i+k*GF->M][j].

Je n'arrive pas à trouver le problème. Si certains d'entre vous ont des idées ?

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
   for (k = 0; k < Code->M ; k++){ 
	   for (i = 0 ; i < GF->M ; i++){
		   for (j = 0 ; j < nbColumn ; j++) {
			   if (Code->H[k][j]== 0){
				   Q[i+k*GF->M][j] = 0;
			   }
			   else{
				   Q[i+k*GF->M][j]= Decoder->LR_INIT[i][Code->H[k][j]];
			   }
		   }
	   }
   }

// correction de la taile de H, ne résoud pas l'erreur.

[jeroman]

Code->M (=252)

J'ai une matrice H de taille GF->M (=8) par nbColumn (=5).

Dans ton code, tu écris :

Code :

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

1
2
3
for (k = 0; k < Code->M ; k++)
(...)
if (Code->H[k][j]== 0)

Or, k va de 0 à Code->M-1, c'est-à-dire 251. Je pense que c'est ça qui provoque le crash.

[SebastienLeto]

Après vérification, je me suis trompé dans les informations que j'ai donné :
H est bien de taille Code->M(=252) par nbColumn(=5) et non ce que j'ai écrit au-dessus.
H contient des entiers.
Q et LR_INIT contienent des floats.

Je vais continuer à chercher l'erreur.

[SebastienLeto]

Voici l'allocation mémoire pour les différentes matrices.
J'ai remarqué la chose suivante :
lorsque les éléments de H sont des short et non des int, l'erreur que j'obtiens à lieu dès la 8ème ligne de Q
lorsque les éléments de H sont des int, l'erreur que j'obtiens n'a lieu qu'à la 787 ligne de Q.
Je me demande si je ne dépasse pas quelque part la taille des matrices. J'ai vérifié cela, mais je ne vois pas d'erreur. Il doit pourtant en avoir une.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
  /* Allocation pour H */
  Code->H = (int**) calloc(Code->M,sizeof(int*));
  if(Code->H == NULL){
	  printf("Error in Allocation of H %s:%d\n",__FILE__,__LINE__);
	  exit(-1);
  }
  Code->H[0] = (int*) calloc(Code->M*Code->maxLine ,sizeof(int));
   if(Code->H[0] == NULL){
	  printf("Error in Allocation of H[0] %s:%d\n",__FILE__,__LINE__);
	  exit(-1);
  }
	for(i = 1; i < Code->M; i++)
		Code->H[i] = Code->H[0] + i*Code->maxLine;

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
 
	/* Allocation pour LR_INIT */
 
  Decoder->LR_INIT = (float**) calloc( Code->N , sizeof(float*));
  if(Decoder->LR_INIT == NULL){
	  printf("Error in Allocation of LR_INIT %s:%d\n",__FILE__,__LINE__);
	  exit(-1);
  }
  Decoder->LR_INIT[0] = (float*) calloc( Code->N * GF->M , sizeof(float));
   if(Decoder->LR_INIT[0] == NULL){
	  printf("Error in Allocation of LR_INIT[0] %s:%d\n",__FILE__,__LINE__);
	  exit(-1);
  }
	for(i = 1; i < Code->N ; i++)
		Decoder->LR_INIT[i] = Decoder->LR_INIT[0] + i* GF->M;
}

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
 
	nbLine = Code->M*GF->M;
	nbColumn = Code->maxLine;
 
	/* Allocation pour Q */
	Q = (float**) calloc(nbLine,sizeof(float*));
  if(Q == NULL){
	  printf("Error in Allocation of Q %s:%d\n",__FILE__,__LINE__);
	  exit(-1);
  }
	Q[0] = (float*) calloc(nbColumn,sizeof(float));
   if(Q[0] == NULL){
	  printf("Error in Allocation of Q[0] %s:%d\n",__FILE__,__LINE__);
	  exit(-1);
   }
   for(i = 1; i < nbLine ; i++){
		Q[i] = Q[0] + i*nbColumn;
   }

[diogene]

Il y a trop peu dans l'allocation de Q

Code :

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

1
2
3
4
5
	Q[0] = (float*) calloc(nbColumn,sizeof(float));
.....
   for(i = 1; i < nbLine ; i++){
		Q[i] = Q[0] + i*nbColumn;
   }

Les pointeurs Q[1], Q[2], .... pointent sur des zones non allouées
N'est-ce pas plutôt

Code :

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

Q[0] = calloc(nbColumn*nbLine,sizeof(float));

[Médinoc]

Tu devrais regrouper ces allocations en fonction, pour être sûr de ne pas faire d'erreur.
Tu peux t'inspirer de cela, en virant le "template", remplaçant T par float, et try/new/catch par calloc:
http://www.developpez.net/forums/d74...e/#post4296674

[SebastienLeto]

Merci Diogène, c'est exactement ça !

Entendu Medinoc, je vais tester ce que tu me conseilles, ça a l'air plutôt pratique.