RE : RE : RE : [Tsp-devel] Malloc et free prob lèmes

[Arquer Stephane]

Salut Frederik,

Effectivement, ça fonctionne mieux.
Les bb_publish ne plantent plus, je tente d'intégrer des bouts de codes de 
l'exemple bb_simu pour lancer bb_provider dans une autre tâche.

Merci pour ton aide.

Je continu.

sa


> -----Message d'origine-----
> De : address@hidden 
> [mailto:address@hidden
> rg] De la part de Frederik Deweerdt
> Envoyé : mercredi 14 novembre 2007 00:08
> À : Transport Sample Protocol development list
> Objet : Re: RE : RE : [Tsp-devel] Malloc et free problèmes
> 
> 
> On Tue, Nov 13, 2007 at 04:51:56PM +0100, Eric Noulard wrote:
> > Le 13/11/07, Arquer Stephane<address@hidden> a écrit :
> > 
> > > > Je n'ai pas dû comprendre ce que tu entends par contexte local. 
> > > > Est-ce que tu pourrais élaborer là-dessus? D'autre part, est-ce 
> > > > que tu es sûr que le bb->name dans posix_*() est bien valide?
> > > Non, le nom n'est pas valide, j'ai créé char *bb_name[]="bb_rtems"
> > 
> > C'est une affection de variable static chaine de caractère. pas 
> > vraiment une création.
> > 
> > Pourrais-tu poster le code actuel de bb_posix_*
> > 
> > > Ensuite la structure de mémorisation des bb utilisés que 
> j'ai créé 
> > > récupère la bonne valeur: bb_used[bb_count].name = "bb_rtems"
> > 
> > je pense qu'il y a mélange entre variables allouées 
> dynamiquement et 
> > statiquement.
> > 
> > une lecture du code permettrait de voir plus clair dans 
> cette affaire 
> > :=)
> OK, j'ai jetté un coup d'oeil, il y a quelques petits 
> problèmes, mais sauf erreur de ma part, la raison du plantage 
> est sans doute le:
> 
>               *bb = (S_BB_T *)malloc(sizeof(shm_size)); <-- 
> sizeof(shm_size) et non shm_size
>               [...]
>               /* Init the freshly allocated structure */
>               memset(*bb, 0, shm_size);
> 
> qui doit aller nullifier la mémoire a gogo.
> 
> Stéphane, est-ce que tu peux appliquer le patch attaché de la 
> manière suivante: cd "racine de tsp"/src/utils/libbb patch 
> -p1 < "nom du patch" voir si ça marche mieux?
> 
> Les autres points mineurs revus par le patch sont:
> - bb_used.name est un tableau de char, pas un tableau de 
> pointeurs de char
> - le success aurait toujours été à TRUE même si on ne trouve 
> pas de bb portant ce nom
> - if (pointeur < 0) est toujours faux, malloc renvoie NULL ou 0
> 
> A bientôt,
> Frederik
>