Discussion :

[phikirua]

Bonjour,
Comment éviter l'erreur de violation d'acces à la ligne I[p][o][q]=(uint16_t)temp;?

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
uint16_t *** get_Images(uint32_t ImageCount,uint32_t ImHeight, uint32_t ImWidth,FILE* fp)
	{
		uint16_t ***I;
		uint16_t temp[2];
		I = (uint16_t ***)malloc((ImHeight+ImWidth+ImageCount)*sizeof(uint16_t**));
		uint16_t tracks[4];
		uint32_t o,p;
		uint16_t q;
                printf("ImWidth %d\n",ImWidth);
		printf("ImWidth*sizeof(uint16_t) %d\n",ImWidth*sizeof(uint16_t));
		printf("ImageCount %d\n",ImageCount);
		printf("ImHeight %d\n",ImHeight);
 
       for (p=0;p<ImageCount;p++)
		{
			for(o=(ImHeight-1);o>=0;o--)
			{
				q=0;
				while(q<ImWidth)
				{
					fread(&temp,2,1,fp);
					//printf("temp %d",temp);
					I[p][o][q]=(uint16_t)temp;
					q=q+2;
				}
			}
			fread(&tracks,sizeof(tracks),1,fp);
 
		}
		return(I);
		free(I);
	}

Message:

'Cine_project.exe'*: Chargé 'C:\Users\qz27p2\Documents\Visual Studio 2010\Projects\Cine_project\Debug\Cine_project.exe', Les symboles ont été chargés.
'Cine_project.exe'*: Chargé 'C:\Windows\SysWOW64\ntdll.dll', Cannot find or open the PDB file
'Cine_project.exe'*: Chargé 'C:\Windows\SysWOW64\kernel32.dll', Cannot find or open the PDB file
'Cine_project.exe'*: Chargé 'C:\Windows\SysWOW64\KernelBase.dll', Cannot find or open the PDB file
'Cine_project.exe'*: Chargé 'C:\Windows\SysWOW64\msvcr100d.dll', Les symboles ont été chargés.
Exception de première chance à 0x00bd3f7d dans Cine_project.exe*: 0xC0000005: Violation d'accès lors de la lecture de l'emplacement 0xcdcdd389.


Merci d'avance pour d'éventuelles réponses.

[Bktero]

C'est moi où tu ne "malloc" pas correctement chacun des dimensions de ta matrice ? --> http://c.developpez.com/faq/?page=L-...a-2-dimensions

C'est quoi ce return dans la boucle ?

C'est quoi cette boucle de free() alors qu'il n'y a pas la boucle de malloc() correspondante ?

[phikirua]

C'est moi où tu ne "malloc" pas correctement chacun des dimensions de ta matrice ? --> http://c.developpez.com/faq/?page=L-...a-2-dimensions

C'est quoi ce return dans la boucle ?

C'est quoi cette boucle de free() alors qu'il n'y a pas la boucle de malloc() correspondante ?

Merci pour ta réponse mais je pense que tu as du mal lire mon code, mon return est à la fin de mes boucles ainsi que mon free, je recopie avec une meilleure indentation mais bon :

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
 
for (p=0;p<ImageCount;p++)
{
	for(o=(ImHeight-1);o>=0;o--)
	{
		q=0;
		while(q<ImWidth)
		{
		        fread(&temp,2,1,fp);
			//printf("temp %d",temp);
			I[p][o][q]=(uint16_t)temp;
			q=q+2;
		}
	}
	fread(&tracks,sizeof(tracks),1,fp);
}
return(I);
free(I);

Je n'avais pas fait attention à mon malloc effectivement (j'ai additionné au lieu de multiplier mes dimensions), si je suis le deuxième exemple du lien que tu m'a envoyé alors je devrais écrire

uint16_t * I = malloc((ImHeight*ImWidth*ImageCount)*sizeof(uint16_t)); ?

A ce moment là ce message apparait: error C2440: 'initialisation'*: impossible de convertir de 'void *' en 'uint16_t *'

Quant à l'autre méthode j'ai écrit cela:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
uint32_t i,j;
uint16_t ***I ;
I = malloc(ImageCount*sizeof(*I));
for(i = 0; i < ImageCount; i++) 
{
	I[i] = malloc(ImHeight * sizeof(**I));
	for(j = 0; j < ImHeight; j++) 
	{
		I[i][j]=malloc(ImWidth* sizeof(***I));
	}
}

Mais pareil probleme de conversion, message du debuger:

1>------ Début de la génération*: Projet*: Cine_project, Configuration*: Debug Win32 ------
1> cine.cpp
1>c:\users\qz27p2\documents\visual studio 2010\projects\cine_project\cine_project\cine.cpp(432): warning C4996: 'fopen': This function or variable may be unsafe. Consider using fopen_s instead. To disable deprecation, use _CRT_SECURE_NO_WARNINGS. See online help for details.
1> c:\program files (x86)\microsoft visual studio 10.0\vc\include\stdio.h(234)*: voir la déclaration de 'fopen'
1>c:\users\qz27p2\documents\visual studio 2010\projects\cine_project\cine_project\cine.cpp(456): error C2440: '='*: impossible de convertir de 'void *' en 'uint16_t ***'
1> La conversion de 'void*' en pointeur vers non 'void' nécessite un cast explicite
1>c:\users\qz27p2\documents\visual studio 2010\projects\cine_project\cine_project\cine.cpp(459): error C2440: '='*: impossible de convertir de 'void *' en 'uint16_t **'
1> La conversion de 'void*' en pointeur vers non 'void' nécessite un cast explicite
1>c:\users\qz27p2\documents\visual studio 2010\projects\cine_project\cine_project\cine.cpp(462): error C2440: '='*: impossible de convertir de 'void *' en 'uint16_t *'
1> La conversion de 'void*' en pointeur vers non 'void' nécessite un cast explicite
========== Génération*: 0 a réussi, 1 a échoué, 0 mis à jour, 0 a été ignoré ==========

Et si je mets les (uint16_t *) ou (uint16_t **) ou (uint16_t ***) je n'ai plus mes problemes de convertion mais j'ai toujours mon probleme de violation d'acces a la ligne
I[p][o][q]=(uint16_t)temp;

[phikirua]

Bon j'ai trouvé la solution.

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
34
35
36
37
38
39
40
41
42
43
44
45
46
 
uint16_t *** get_Images(uint32_t ImageCount,uint32_t ImHeight, uint32_t ImWidth,FILE* fp)
{
		uint16_t temp;
		uint32_t i,j;
		uint16_t ***I ;
		I = (uint16_t ***)malloc(ImageCount*sizeof(*I));
		for(i = 0; i < ImageCount; i++) 
		{
			I[i] = (uint16_t **)malloc(ImHeight * sizeof(**I));
			for(j = ImHeight; j >0; j--) 
			{
				I[i][j]=(uint16_t *)malloc(ImWidth * sizeof(***I));
			}
		}
		uint16_t tracks[4];
		uint32_t o,p;
		uint32_t q;
        for (p=0;p<ImageCount;p++)
		{
			for(o=ImHeight;o>0;o--)
			{
				for(q=0;q<ImWidth;q++)
				{
					fread(&temp,2,1,fp);
					I[p][o][q]=temp;		
				}
 
			}
			fread(&tracks,sizeof(tracks),1,fp);
		}
 
		return(I);
		for(i=0 ; i < ImageCount ; i++)
		{
			for(j=ImHeight ; j >0 ; j--)
			{
					free(I[i][j]);
			}
		}
		for(i=0 ; i < ImageCount ; i++)
		{
			free(I[i]);
		}
		free(I);
	}

J'avais mal défini ma boucle pour décrémenter et j'avais incrémenter ma boucle dans malloc, du coup ca fonctionnait pas.

[Bktero]

Je me suis en effet totalement basé sur l'indentation, pardon. Encore une fois, on voit l'importance de l'indentation !

Content que tu aies trouvé la solution à ton problème. Bonne continuation.