fuite de mémoire!

fuite de mémoire! - C++ - Programmation

Marsh Posté le 16-08-2005 à 20:05:34    

Salut!
Je programme un jeu de combat avec OpenGl et Glut et lorsque j'appelle cette fonction, le gestionnaire des taches de windows m'indique que la mémoire utilisée par mon processus grimpe en flèche (je passe environ de 50 Mo à 120 Mo):
 

Code :
  1. using namespace std;
  2. #include <iostream>
  3. #include <direct.h>
  4. #include "baston.hpp"
  6. extern GLubyte tex_ciel[2048*2048*4], tex_sol[2048*2048*4];
  7. extern GLubyte tex_devant[1024*1024*4], tex_derriere[1024*1024*4];
  8. extern GLubyte tex_droite[1024*1024*4],tex_gauche[1024*1024*4];
  10. extern GLuint textures_skybox[6];
  12. extern int nb_mondes;
  13. extern int monde_en_cours;
  15. //------------------------------------------------------------------------------
  16. //-------------------------INITIALISATION DE LA SKYBOX--------------------------
  17. //------------------------------------------------------------------------------
  18. void init_skybox()
  19. {
  20.     int i,j,k;
  21.     FILE *fich;
  22.     char nomonde[3];
  23.     char monde[15];
  24.    
  25.     GLubyte en_tete[37];
  26.    
  27.     glEnable(GL_TEXTURE_2D);     
  28.     glGenTextures(6,textures_skybox);
  29.    
  30.    
  31.     //Répertoires pour les textures
  32.     strcpy(monde,"monde" );
  33.     to_char(monde_en_cours,monde+5);//ajoute le numéro du monde à la fin de monde
  34.    
  35.  
  36.     //Choix du répertoire en cours
  37.     chdir(monde);
  38.      
  39.     //ciel:
  40.     fich=fopen("ciel.bmp","rb" ); 
  41.     if(fich==NULL) exit(0);
  42.    
  43.     fread (en_tete,1,54,fich);
  44.    
  45.     for(i=0;i<2048;i++)
  46.     {
  47.         for(j=0;j<2048;j++)
  48.         {
  49.             fread(&tex_ciel[(i*2048+j)*4+2],sizeof(GLubyte),1,fich);
  50.             fread(&tex_ciel[(i*2048+j)*4+1],sizeof(GLubyte),1,fich);
  51.             fread(&tex_ciel[(i*2048+j)*4+0],sizeof(GLubyte),1,fich);
  52.             tex_ciel[(i*2048+j)*4+3]=255;
  53.         }
  54.     }     
  55.     glBindTexture(GL_TEXTURE_2D,textures_skybox[0]);
  56.     glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_MIN_FILTER, GL_NEAREST);
  57.     glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_MAG_FILTER, GL_NEAREST);
  58.     glTexParameteri(GL_TEXTURE_2D,GL_TEXTURE_WRAP_S,GL_REPEAT );
  59.     glTexParameteri(GL_TEXTURE_2D,GL_TEXTURE_WRAP_T,GL_REPEAT );
  60.     glTexImage2D(GL_TEXTURE_2D,0,GL_RGBA,2048,2048,0,GL_RGBA,GL_UNSIGNED_BYTE,tex_ciel);
  61.    
  62.     //sol:
  63.     fich=fopen("sol.bmp","rb" );
  64.     if(fich==NULL) exit(0);
  65.    
  66.     fread (en_tete,1,54,fich);
  67.    
  68.     for(i=0;i<2048;i++)
  69.     {
  70.         for(j=0;j<2048;j++)
  71.         {
  72.             fread(&tex_sol[(i*2048+j)*4+2],sizeof(GLubyte),1,fich);
  73.             fread(&tex_sol[(i*2048+j)*4+1],sizeof(GLubyte),1,fich);
  74.             fread(&tex_sol[(i*2048+j)*4+0],sizeof(GLubyte),1,fich);
  75.             tex_sol[(i*2048+j)*4+3]=255;
  76.         }
  77.     }
  78.    
  79.     fclose (fich); 
  80.     glBindTexture(GL_TEXTURE_2D,textures_skybox[1]);
  81.     glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_MIN_FILTER, GL_NEAREST);
  82.     glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_MAG_FILTER, GL_NEAREST);
  83.     glTexParameteri(GL_TEXTURE_2D,GL_TEXTURE_WRAP_S,GL_REPEAT );
  84.     glTexParameteri(GL_TEXTURE_2D,GL_TEXTURE_WRAP_T,GL_REPEAT );
  85.     glTexImage2D(GL_TEXTURE_2D,0,GL_RGBA,2048,2048,0,GL_RGBA,GL_UNSIGNED_BYTE,tex_sol);
  86.    
  87.     //derriere:
  88.     fich=fopen("derriere.bmp","rb" );
  89.     if(fich==NULL) exit(0);
  90.    
  91.     fread (en_tete,1,54,fich);
  92.    
  93.     for(i=0;i<1024;i++)
  94.     {
  95.         for(j=0;j<1024;j++)
  96.         {
  97.             fread(&tex_derriere[(i*1024+j)*4+2],sizeof(GLubyte),1,fich);
  98.             fread(&tex_derriere[(i*1024+j)*4+1],sizeof(GLubyte),1,fich);
  99.             fread(&tex_derriere[(i*1024+j)*4+0],sizeof(GLubyte),1,fich);
  100.             tex_derriere[(i*1024+j)*4+3]=255;
  101.         }
  102.     }
  103.    
  104.     fclose (fich); 
  105.     glBindTexture(GL_TEXTURE_2D,textures_skybox[2]);
  106.     glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_MIN_FILTER, GL_NEAREST);
  107.     glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_MAG_FILTER, GL_NEAREST);
  108.     glTexParameteri(GL_TEXTURE_2D,GL_TEXTURE_WRAP_S,GL_REPEAT );
  109.     glTexParameteri(GL_TEXTURE_2D,GL_TEXTURE_WRAP_T,GL_REPEAT );
  110.     glTexImage2D(GL_TEXTURE_2D,0,GL_RGBA,1024,1024,0,GL_RGBA,GL_UNSIGNED_BYTE,tex_derriere);
  111.    
  112.     //devant:
  113.     fich=fopen("devant.bmp","rb" );
  114.     if(fich==NULL) exit(0);
  115.    
  116.     fread (en_tete,1,54,fich);
  117.    
  118.     for(i=0;i<1024;i++)
  119.     {
  120.         for(j=0;j<1024;j++)
  121.         {
  122.             fread(&tex_devant[(i*1024+j)*4+2],sizeof(GLubyte),1,fich);
  123.             fread(&tex_devant[(i*1024+j)*4+1],sizeof(GLubyte),1,fich);
  124.             fread(&tex_devant[(i*1024+j)*4+0],sizeof(GLubyte),1,fich);
  125.             tex_devant[(i*1024+j)*4+3]=255;
  126.         }
  127.     }
  128.    
  129.     fclose (fich); 
  130.     glBindTexture(GL_TEXTURE_2D,textures_skybox[3]);
  131.     glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_MIN_FILTER, GL_NEAREST);
  132.     glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_MAG_FILTER, GL_NEAREST);
  133.     glTexParameteri(GL_TEXTURE_2D,GL_TEXTURE_WRAP_S,GL_REPEAT );
  134.     glTexParameteri(GL_TEXTURE_2D,GL_TEXTURE_WRAP_T,GL_REPEAT );
  135.     glTexImage2D(GL_TEXTURE_2D,0,GL_RGBA,1024,1024,0,GL_RGBA,GL_UNSIGNED_BYTE,tex_devant);
  136.    
  137.     //gauche:
  138.     fich=fopen("gauche.bmp","rb" );
  139.     if(fich==NULL) exit(0);
  140.    
  141.     fread (en_tete,1,54,fich);
  142.    
  143.     for(i=0;i<1024;i++)
  144.     {
  145.         for(j=0;j<1024;j++)
  146.         {
  147.             fread(&tex_gauche[(i*1024+j)*4+2],sizeof(GLubyte),1,fich);
  148.             fread(&tex_gauche[(i*1024+j)*4+1],sizeof(GLubyte),1,fich);
  149.             fread(&tex_gauche[(i*1024+j)*4+0],sizeof(GLubyte),1,fich);
  150.             tex_gauche[(i*1024+j)*4+3]=255;
  151.         }
  152.     }
  153.    
  154.     fclose (fich); 
  155.     glBindTexture(GL_TEXTURE_2D,textures_skybox[4]);
  156.     glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_MIN_FILTER, GL_NEAREST);
  157.     glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_MAG_FILTER, GL_NEAREST);
  158.     glTexParameteri(GL_TEXTURE_2D,GL_TEXTURE_WRAP_S,GL_REPEAT );
  159.     glTexParameteri(GL_TEXTURE_2D,GL_TEXTURE_WRAP_T,GL_REPEAT );
  160.     glTexImage2D(GL_TEXTURE_2D,0,GL_RGBA,1024,1024,0,GL_RGBA,GL_UNSIGNED_BYTE,tex_gauche); 
  161.    
  162.     //droite:
  163.     fich=fopen("droite.bmp","rb" );
  164.     if(fich==NULL) exit(0);
  165.    
  166.     fread (en_tete,1,54,fich);
  167.    
  168.     for(i=0;i<1024;i++)
  169.     {
  170.         for(j=0;j<1024;j++)
  171.         {
  172.             fread(&tex_droite[(i*1024+j)*4+2],sizeof(GLubyte),1,fich);
  173.             fread(&tex_droite[(i*1024+j)*4+1],sizeof(GLubyte),1,fich);
  174.             fread(&tex_droite[(i*1024+j)*4+0],sizeof(GLubyte),1,fich);
  175.             tex_droite[(i*1024+j)*4+3]=255;
  176.         }
  177.     }
  178.    
  179.     fclose (fich); 
  180.     glBindTexture(GL_TEXTURE_2D,textures_skybox[5]);
  181.     glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_MIN_FILTER, GL_NEAREST);
  182.     glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_MAG_FILTER, GL_NEAREST);
  183.     glTexParameteri(GL_TEXTURE_2D,GL_TEXTURE_WRAP_S,GL_REPEAT );
  184.     glTexParameteri(GL_TEXTURE_2D,GL_TEXTURE_WRAP_T,GL_REPEAT );
  185.     glTexImage2D(GL_TEXTURE_2D,0,GL_RGBA,1024,1024,0,GL_RGBA,GL_UNSIGNED_BYTE,tex_droite);
  186.    
  187.    
  188.     chdir(".." );
  189. }


 
Pourtant je n'ai aucune allocation dynamique puisque mes tableaux ont des tailles fixes et sont déclarés variables globales dans le .cpp principal
 
merci


---------------
deluser --remove-home ptitchep
Reply

Marsh Posté le 16-08-2005 à 20:05:34   

Reply

Marsh Posté le 16-08-2005 à 20:47:35    

#  for(i=0;i<2048;i++)
#     {
#         for(j=0;j<2048;j++)
#         {
#             fread(&tex_ciel[(i*2048+j)*4+2],sizeof(GLubyte),1,fich);
#             fread(&tex_ciel[(i*2048+j)*4+1],sizeof(GLubyte),1,fich);
#             fread(&tex_ciel[(i*2048+j)*4+0],sizeof(GLubyte),1,fich);
#             tex_ciel[(i*2048+j)*4+3]=255;
#         }
#     }        
 
 
c'est idiot tout ça, tu veux faire tout en un seul fread. idem pour le reste. beurk les variables globales aussi grosse. avec des BMP, tu peux meme faire du mappage mémoire directe :)

Reply

Marsh Posté le 16-08-2005 à 20:49:19    

GLubyte en_tete[37];
 
...
 
 fread (en_tete,1,54,fich);
 
 
perdu

Reply

Marsh Posté le 16-08-2005 à 20:50:55    

Je ne sais pas si les lignes suivantes sont la cause du problème, mais en tous cas, il y a là un petit bug.

  GLubyte en_tete[37];  
 ...
    fread (en_tete,1,54,fich);  


Il faudrait que "en_tete" soit déclaré avec au moins 54 octets, sinon il y a un débordement sur les zones mémoires voisines. :)
 
Edit : T'es trop rapide, Taz !


Message édité par olivthill le 16-08-2005 à 20:52:09
Reply

Marsh Posté le 17-08-2005 à 12:13:08    

Sinon une fois que tu as instancié tes textures, ça sert a rien de garder une copie des images. Donc tu peux abandonner tes gros tableaux globaux, utiliser un tableau dynamique assez grand (pour réutiliser le meme pour toutes les faces) au moment du loading et le virer proprement à la fin du Init.

Reply

Marsh Posté le 17-08-2005 à 13:47:45    

Ok je croyais que openGl se servait de mes tableaux pour les textures par la suite.
Pour le fread 54 dans un tableau de 37  c'est une petite erreur c'est 0x37 dsl. (j'ai compté l'en-tete dans mon éditeur hexa et forcément j'ai l'offset en hexa...
Tout s'arrange merci. En fait je ne savais pas qu'OpenGl chargait mes textures en mem et du coup je me souçiais pas des textures. Erreur fatale, il manquait:
glDeleteTextures(6,textures_skybox);
d'où la fuite de memoire.
Merci bien!!!


---------------
deluser --remove-home ptitchep
Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed