Discussion :

[Ph.denis]

Salut à tous,

j'ai une erreur lors de l'éxécution de mon programme et j'ai déjà tout fait pour la trouver mais rien n'y fait je suis bloqué...

Voici la partie de code "foireuse" :

Code C :

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

1
2
3
4
5
6
7
8
9
10
 
printf("Correspondance : %d\n",numberPointCoresp);
Container* tmpModel = NULL;
printf("ici1.1111555\n");
tmpModel = malloc(sizeof(Container));
printf("ici1.1111\n");
for(int a = 0;a < NBR_POINT;a++) 
	tmpModel->inliers[a] = 0;
 
printf("ici1.1\n");

et voici l'erreur lors de l'exécution :

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
Correspondance : 669
ici1.1111555
*** glibc detected *** ./Vision: corrupted double-linked list: 0x086e7b80 ***
======= Backtrace: =========
/lib/tls/i686/cmov/libc.so.6[0xb7cd03f9]
/lib/tls/i686/cmov/libc.so.6[0xb7cd21bd]
/lib/tls/i686/cmov/libc.so.6(__libc_malloc+0x90)[0xb7cd3fc0]
./Vision[0x8049237]
./Vision[0x804b649]
/lib/tls/i686/cmov/libc.so.6(__libc_start_main+0xe0)[0xb7c7e050]
./Vision[0x8048b31]
======= Memory map: ========
08048000-0804d000 r-xp 00000000 08:04 102659     /home/philippe/Desktop/Vision_Harris_Homographie/Debug/Vision
0804d000-0804e000 rw-p 00004000 08:04 102659     /home/philippe/Desktop/Vision_Harris_Homographie/Debug/Vision
0804e000-0a3da000 rw-p 0804e000 00:00 0          [heap]
b1147000-b65f0000 rw-p b1147000 00:00 0 
b6700000-b6721000 rw-p b6700000 00:00 0 
b6721000-b6800000 ---p b6721000 00:00 0 
b6896000-b68e2000 rw-p b6896000 00:00 0 
b6a10000-b6a2f000 r--p 00000000 08:01 410131     /usr/share/locale-langpack/fr/LC_MESSAGES/libc.mo
b6a2f000-b6a41000 r-xp 00000000 08:01 131155     /usr/lib/gtk-2.0/2.10.0/engines/libubuntulooks.so
b6a41000-b6a42000 rw-p 00011000 08:01 131155     /usr/lib/gtk-2.0/2.10.0/engines/libubuntulooks.so
b6a42000-b6a67000 r--p 00000000 08:01 407464     /usr/share/locale-langpack/fr/LC_MESSAGES/gtk20-properties.mo
b6a67000-b6a70000 r-xp 00000000 08:01 1777218    /lib/tls/i686/cmov/libnss_files
...

Je n'ai pas affiché toute la trace, si vous estimez qu'elle pourrait vous être utile pour répondre à ma question n'hésitez pas à me la demander.

J'espère que vous allez pouvoir me debloquer...

Un tout grand merci d'avance,

Philippe

[valefor]

Comment est déclarée ta structure container ?

[Arnaud F.]

Salut,

ne serait-ce pas :

Code C :

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

tmpModel = malloc(sizeof(Container));

Il faut rajouter une *, ce qui donne :

Code C :

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

tmpModel = malloc(sizeof(Container*));

Non?

[Ph.denis]

Code :

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

1
2
3
4
5
6
typedef struct Container_t {
	int inliers[NBR_POINT];
	int nbrInliers;
	CvMat* homography;
	struct Container_t* next;
} Container;

Ce qui est bizarre c'est que le code fonctionne dans une boucle...donc cela fonctionne bien puis à un moment y'a une erreur sur ce code...

Buchs : je ne pense pas que cela soit ça

[aoyou]

Ce code ne me paraît pas suffisant pour comprendre ce qui se passe.
Pourrais-tu faire un programme minimaliste ayant le même problème ?

printf étant bufferisé, tu n'es pas sûr d'afficher le dernier message. Après chacun de tes printf, flushe stdout (fflush (stdout); ) ou compile en debug et utilise un debugger pour réellement savoir où ça plante.

Au passage, il vaut mieux faire

Code :

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

 tmpModel = malloc(sizeof *tmpModel);

Tu pourras changer le type de tmpModel sans surprise par la suite.

[souviron34]

printf étant bufferisé, tu n'es pas sûr d'afficher le dernier message. Après chacun de tes printf, flushe stdout (fflush (stdout); ) ou compile en debug et utilise un debugger pour réellement savoir où ça plante.

ou plus simple

Code :

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

fprintf ( stderr, "....");

qui est synchrone..... et qui est fait pour...

[aoyou]

Hum, je ne dirais pas fait pour. Il permet en premier lieu de séparer les erreurs du flux de sortie standard. Mais oui tu peux aussi utiliser fprintf (stderr, ...) qui a l'avantage de ne pas être bufferisé. Mais personnellement, rien ne vaut un bon vieux debugger pour savoir tout de suite où ça plante...

[hegros]

si tu as un code qui est compilable cela peut aider

[Ph.denis]

j'ai trouvé...
À un moment donné j'allais trop loin avec un pointeur

[inh40]

Tu pourrais un peu plus détailler ta correction stp ? Car j'ai un problème similaire.

[hegros]

[QUOTE=Ph.denis;2963033]Salut à tous,

Code C :

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

1
2
3
4
5
6
7
8
9
10
 
printf("Correspondance : %d\n",numberPointCoresp);
Container* tmpModel = NULL;
printf("ici1.1111555\n");
tmpModel = malloc(sizeof(Container));
printf("ici1.1111\n");
for(int a = 0;a < NBR_POINT;a++) 
	tmpModel->inliers[a] = 0;
 
printf("ici1.1\n");

si inliers est un membre de type pointeur il faut faire également l'allocation. la question est comment déclares-tu inliers

Code :

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

1
2
3
4
5
6
 
struct Container{
  int *inliers;
//ou
int inliers[NBR_POINT];
};

[contremaitre]

Tu pourrais un peu plus détailler ta correction stp ? Car j'ai un problème similaire.

Il avait un problème de mémoire apparemment, il lisait ou écrivait dans une zone mémoire invalide, donc vérifie si tu ne fait pas ce genre d'erreur.

[Emmanuel Delahaye]

Voici la partie de code "foireuse" :

Ton code est incomplet et ne peut être testé. Ceci fonctionne (merci pour les 15 minutes de rétro-ingéniérie...) :

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
 
#include <stdio.h>
#include <stdlib.h>
 
#define NBR_POINT 10
 
typedef struct
{
   int inliers[NBR_POINT];
}
Container;
 
int main (void)
{
   Container *tmpModel = NULL;
   printf ("%s:%d\n", __FILE__, __LINE__);
   tmpModel = malloc (sizeof (Container));
   printf ("%s:%d\n", __FILE__, __LINE__);
   {
      int a;
      for (a = 0; a < NBR_POINT; a++)
      {
         tmpModel->inliers[a] = 0;
      }
   }
   printf ("%s:%d\n", __FILE__, __LINE__);
   return 0;
}

Code :

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

1
2
3
4
5
6
 
main.c:15
main.c:17
main.c:25
 
Press ENTER to continue.

[contremaitre]

Ton code est incomplet et ne peut être testé. Ceci fonctionne (merci pour les 15 minutes de rétro-ingéniérie...) :

le PO a résolut son problème depuis le 19/02/2008

[Emmanuel Delahaye]

le PO a résolut son problème depuis le 19/02/2008

Oui, mais visiblement, quelqu'un (inh40) a besoin de précision...