Segmentation Fault sur FillRect

Bonjour bonjour :)

Comme mentionné dans le titre je rencontre un souci lors d'un appel a SDL_FillRect.
Les circonstances sont les suivantes:
Un tableau de SDL_Surface (qui va contenir les images d'un menu graphique) est déclaré:

Code:

---

SDL_Surface **bmpMenu = NULL;

---

Puis le programme fait un appel à une fonction chargerMenu déclarée et fonctionnant comme suit:

Code:

---

SDL_Surface **chargerMenu(SDL_Surface **bmpMenu);

---

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23

SDL_Surface **chargerMenu(SDL_Surface **bmpMenu) { bmpMenu = malloc(3*sizeof(SDL_Surface*));   bmpMenu[0] = SDL_CreateRGBSurface(SDL_HWSURFACE, 50, 50, 32, 0, 0, 0, 0); bmpMenu[0]->clip_rect.x = 200; bmpMenu[0]->clip_rect.y = 100;   bmpMenu[1] = SDL_CreateRGBSurface(SDL_HWSURFACE, 50, 50, 32, 0, 0, 0, 0); bmpMenu[1]->clip_rect.x = 200; bmpMenu[1]->clip_rect.y = 200;   bmpMenu[2] = SDL_CreateRGBSurface(SDL_HWSURFACE, 50, 50, 32, 0, 0, 0, 0); bmpMenu[2]->clip_rect.x = 200; bmpMenu[2]->clip_rect.y = 300;   int i; for (i=0; i<3; i++) if (bmpMenu[i] == NULL) printf("Souci d'allocation pour la %d surface\n", i);   return bmpMenu; }

---

Cette fonction alloue de la mémoire avec malloc pour 3 pointeurs sur SDL_Surface, puis alloue de la mémoire pour une SDL_Surface avec SDL_CreateRGBSurface pour chacun des 3 pointeurs.
Les coordonnées des surfaces sont stockées directement dans clip_rect.

Finalement le programme appelle une fonction colleMenu définie comme ci-dessous:

Code:

---

void colleMenu(SDL_Surface **bmpMenu, SDL_Surface *ecran;)

---

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20

void colleMenu(SDL_Surface **bmpMenu, SDL_Surface *ecran) { SDL_Rect pos;   SDL_Delay(2000); SDL_FillRect(ecran, NULL, SDL_MapRGB(ecran->format, 50, 50, 50)); SDL_Flip(ecran);   int i; for (i=0; i<3; i++) { SDL_Delay(2000); pos.x = bmpMenu[i]->clip_rect.x; // Accès en lecture: OK pos.y = bmpMenu[i]->clip_rect.y; bmpMenu[i]->clip_rect.x = 123; // Test d'accès en écriture: OK SDL_FillRect(bmpMenu[i], NULL, SDL_MapRGB(ecran->format, 255, 0, 0)); // Segmentation Fault pour i=1 SDL_BlitSurface(bmpMenu[i], NULL, ecran, &pos); SDL_Flip(ecran);   } SDL_Flip(ecran); }

---

Les SDL_Delay servent a temporiser le programme pour observer son exécution.
La surface principale, ecran, est remplie avec du gris, elle s'affiche bien a l'écran.
Puis je lance une boucle for qui va récupérer les coordonnées de chaque surface stockées dans clip_rect, appliquer une couleur rouge puis coller cette surface et rafraichir l'écran.
Comme indiquée en commentaire, les coordonnées récupérées dans pos sont valides et il est toujours possible de modifier les coordonnées originales des surfaces.

Seulement la suite ne se déroule pas tout a fait comme voulu:

• lors de la première itération, le carré correspondant a bmpMenu[0] s'affiche au bon endroit mais il est noir.
• lors du deuxième passage dans la boucle, SDL_FillRect provoque une erreur de segmentation (pour bmpMenu[1] donc).
• aucune surface n'est NULL.

Détail: lorsque une couleur (marron) était appliquée dans chargerMenu puis remplacée par une autre (rouge) dans colleMenu, le premier carré, bmpMenu[0] restait marron.

Voici les 3 indices dont je dispose actuellement, les suspicions que j'ai se portent sur:

• mauvaise utilisation des tableaux de SDL_Surface (fort probable)
• autre erreur (bien probable)
• incohérence au niveau du paramètre SDL_PixelFormat* demandé par SDL_MapRGB pour générer les couleurs (probable)

A savoir, ces 2 fonctions sont appelées successivement dans la même fonction:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

void menu(int *continuer, SDL_Surface *ecran) { SDL_Surface **bmpMenu = NULL;   bmpMenu = chargerMenu(bmpMenu);   colleMenu(bmpMenu, ecran);   ...   libereMenu(bmpMenu); }

---

Je pense que mon code n'est pas bien compliqué pour toute personne ayant déjà manipulé la SDL (et le C) et qu'un regard neuf sur celui-ci pourrait m'apporter quelques éclaircissements, donc merci d'avance pour toute réponse :)

EDIT: La solution a été donnée sur un autre forum:
Ne jamais modifier les variables contenues dans clip_rect
Comme indiqué sur la doc de la SDL
Je ne comprends tout de même pas pourquoi cela provoque un plantage, si quelqu'un a une explication je suis preneur :)
Quoiqu'il en soit: problème résolu :P

Réponse: regarder dans la source:

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 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61

/*  * Set the clipping rectangle for a blittable surface  */ SDL_bool SDL_SetClipRect(SDL_Surface *surface, const SDL_Rect *rect) { SDL_Rect full_rect;   /* Don't do anything if there's no surface to act on */ if ( ! surface ) { return SDL_FALSE; }   /* Set up the full surface rectangle */ full_rect.x = 0; full_rect.y = 0; full_rect.w = surface->w; full_rect.h = surface->h;   /* Set the clipping rectangle */ if ( ! rect ) { surface->clip_rect = full_rect; return 1; } return SDL_IntersectRect(rect, &full_rect, &surface->clip_rect); }   /*  * A function to calculate the intersection of two rectangles:  * return true if the rectangles intersect, false otherwise  */ static __inline__ SDL_bool SDL_IntersectRect(const SDL_Rect *A, const SDL_Rect *B, SDL_Rect *intersection) { int Amin, Amax, Bmin, Bmax;   /* Horizontal intersection */ Amin = A->x; Amax = Amin + A->w; Bmin = B->x; Bmax = Bmin + B->w; if(Bmin > Amin) Amin = Bmin; intersection->x = Amin; if(Bmax < Amax) Amax = Bmax; intersection->w = Amax - Amin > 0 ? Amax - Amin : 0;   /* Vertical intersection */ Amin = A->y; Amax = Amin + A->h; Bmin = B->y; Bmax = Bmin + B->h; if(Bmin > Amin) Amin = Bmin; intersection->y = Amin; if(Bmax < Amax) Amax = Bmax; intersection->h = Amax - Amin > 0 ? Amax - Amin : 0;   return (intersection->w && intersection->h); }

---

En gros SDL_SetClipRect fait en sorte à chaque fois que le rectangle ne dépasse pas les limites de la surface.

Toi, en changeant rect.x et rect.y à 200, ne change pas la largeur et la hauteur de clip_rect. Comme par défaut c'est la largeur et la hauteur de la surface, ça te fait un rectangle qui va en dehors des limites de la surface, et donc quand tu le remplis avec SDL_FillRect, ça peut faire une segfault. Il faut changer la largeur et la hauteur à la main aussi, si tu changes x et y.

Aussi, qu'est ce qu'il te prend de mettre 200 en coordonnées à une surface qui ne fait que 50 en hauteur et largeur? oO

Enfin bref j'espère que tu as ton explication.