IdentifiantMot de passe
Loading...
Mot de passe oublié ?Je m'inscris ! (gratuit)
Navigation

Inscrivez-vous gratuitement
pour pouvoir participer, suivre les réponses en temps réel, voter pour les messages, poser vos propres questions et recevoir la newsletter

Bibliothèque standard C Discussion :

Probleme de free()


Sujet :

Bibliothèque standard C

Vue hybride

Message précédent Message précédent   Message suivant Message suivant
  1. #1
    Membre averti
    Profil pro
    Inscrit en
    Janvier 2006
    Messages
    31
    Détails du profil
    Informations personnelles :
    Localisation : France

    Informations forums :
    Inscription : Janvier 2006
    Messages : 31
    Par défaut Probleme de free()
    BOnjour, je suis en licence pro et je cherche une aide pour une erreur que je rencontre.

    VOila j'ai une fonction main qui est le menu de mon programme
    mais lors du free tout en bas j'ai une erreur de free invalid !

    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
    Int main (int argc, char *argv[]){
     
    	int repMenu;
    	char* chemSource;
    	char* chemDesti;
     
    	//menu
    	printf("\n");
    	printf("***************************************************\n");
    	printf("|                                                 |\n");
    	printf("|    1-Pour une synchro à partir du PC source	  |\n");
    	printf("|                                                 |\n");
    	printf("|   2-Pour une synchro vers le PC de destination  |\n");
    	printf("|                                                 |\n");
    	printf("***************************************************\n");
    	printf("\nVotre choix : ");
    	scanf("%d",&repMenu);
     
    	//gestion des chemins pour qu'il finissent toujours par un "/" meme si le caractère n'est pas renseigné
    	chemSource=gererSlash(argv[1]);
    	chemDesti=gererSlash(argv[2]);
     
    	//Lancement du programme en fonction du choix de l'utilisateur
    	switch (repMenu){
    		case 1:
    			utilisationPCA(chemSource,chemDesti);
    			break;
    		case 2:
    			utilisationPCB(chemSource,chemDesti);
    			break;
    	}
     
    	//liberation de la mémoire
            printf("%s\n",chemSource); //ça ça marche !!
    	free(chemSource);
    	free(chemDesti);
     
    	//fin du programme
    	return 0;
    }
    dabbord la fonction gererSlash:
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
    5
    6
    7
    8
    9
    10
    char * gererSlash(char *chemin){
    	char * tmp=NULL;
    	if ( ( chemin[strlen(chemin)-1] )!='/' ){
    		tmp=malloc( (strlen(chemin)+2)*sizeof(char)  );
    		strcpy(tmp,chemin);
    		strcat(tmp,"/");
    		return tmp;
    	}
    	return chemin;
    }
    Je lance le cas 2:
    qui lance utilisationPCB
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
    5
    6
    7
    8
    9
    10
    11
    12
    13
    void utilisationPCB(char *chemRep,char *chemDesti){
     
    	//declaration d'une chaine de fichier vide
    	chaineFichier fichiersPresent=NULL;
     
    	//desarchivage de l'archive présente sur la clé
    	if(desarchiver(chemRep,chemDesti)==-1)
    		fprintf(stderr,"Impossible de continuer !\n");
    	else{
    		parcourirRep(chemDesti, NULL,&fichiersPresent);
    		creerFichierSync(chemRep,fichiersPresent);
    	}
    }
    Dans cette fonction 2 autres sous fonctions :
    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
    void creerFichierSync(char* chemDesti,chaineFichier fichiersAEcrire){
    	FILE *fichierSync;
    	char *chemFichierSync;
     
     
    	//creation du fichier de synchro
    	chemFichierSync=malloc((strlen(chemDesti)+17)*sizeof(char));
    	strcpy(chemFichierSync,chemDesti);
    	strcat(chemFichierSync,"/FichierSync.txt");
    	fichierSync=fopen (chemFichierSync, "w");
    	while (fichiersAEcrire!=NULL){
    		fprintf(fichierSync,"%s",fichiersAEcrire->cheminFichier);
    		fputs(";",fichierSync);
    		fprintf(fichierSync,"%d",*(fichiersAEcrire->date));
    		fputs("\n",fichierSync);
    		fichiersAEcrire=fichiersAEcrire->suivant;
    	}
    	//on a plus besoin du chemin du rep donc on free chemRep
    	//free(chemFichierSync);
    	//on a plus besoin du chemin relatif donc on free chemRElatif
    	//free(chemRelatif);
    	fclose(fichierSync);
    }
    Et :
    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
    47
    48
    49
    50
    51
    52
    53
    54
    55
    56
    57
    58
    59
    60
    61
    62
    63
    64
    65
    66
    67
    68
    69
    70
    71
    72
    73
    74
    75
    76
    77
    78
    79
    80
    81
    82
    83
    84
    85
    86
    87
    88
    89
    90
    91
    92
    93
    94
    95
    96
    97
    98
    99
    100
    101
    102
    103
    void parcourirRep(char *chemRep, char *chemRelatif,chaineFichier *fichiersPresent){
     
    	struct dirent *entree;
    	struct stat statEntree;
    	char *cheminComplet=NULL;
    	char *chemRelatifBak=NULL;
    	char *chemEtFichier=NULL;
    	DIR *rep;
     
    	//si le chemin relatif est différent de NULL, c'est à dire si on est rentré dans un dossier à partir du dossier d'origine.
    	if (chemRelatif!=NULL){
    		//on concatène le chemin de base au chemin du repertoire fils pour avoir le chemin complet.
    		cheminComplet=malloc((strlen(chemRep)+strlen(chemRelatif)+1)*sizeof(char));
    		strcpy(cheminComplet,chemRep);
    		//strcat(cheminComplet,"/");
    		strcat(cheminComplet,chemRelatif);
    	}
    	//si on est dans le repertoire racine.
    	else{
    		//le chemin complet est le chemin racine de base.
    		cheminComplet=malloc((strlen(chemRep)+1)*sizeof(char));
    		strcpy(cheminComplet,chemRep);
    		//strcat(cheminComplet,"/");
    	}
    	printf("chemin : %s\n",cheminComplet);
    	//on ouvre ce repertoire.
    	rep = opendir(cheminComplet);
     
    	//si le repertoire n'a pu etre ouvert => message d'erreur
    	if (rep==NULL){
    		fprintf(stderr,"%s : Problème lors de l'ouverture !\n", chemRep);
    	}
    	//si on arrive a ouvrir le repertoire.
    	else{
    		//on boucle sur toutes les entrees du repertoire
    		while ((entree = readdir(rep))){
     
    			//si l'entree est un repertoire autre que '.' et '..'
    			if (entree->d_type==4 && strcmp(entree->d_name,".")!=0 && strcmp(entree->d_name,"..")!=0){
    				//si le chemin relatif n'est pas null
    				if (chemRelatif!=NULL){
    					//on alloue une place mémoire pour sauvegarder l'ancien chemin relatif et on le copie dedans
    					chemRelatifBak=malloc((strlen(chemRelatif)+1)*sizeof(char));
    					strcpy(chemRelatifBak,chemRelatif);
    					//on réallou la taille mémoire pour le cheminRElatif
    					chemRelatif=realloc(chemRelatif,(strlen(chemRelatif)+strlen(entree->d_name)+1)*sizeof(char));
    					strcat(chemRelatif,entree->d_name);
    					strcat(chemRelatif,"/");
    				}
    				else{
    					//le chemin est null donc 
    					//on allou une place mémoire pour le chemin relatif car c'est la premiere fois qu'il est crée
    					chemRelatif=malloc((strlen(entree->d_name)+1)*sizeof(char));
    					strcpy(chemRelatif,entree->d_name);
    					strcat(chemRelatif,"/");
    				}
    				//on relance récursivement la fonction pour aller chercher dans ce repertoire.
    				parcourirRep(chemRep,chemRelatif,&(*fichiersPresent));
    				//on récupère l'ancien cheminRelatif
    				chemRelatif=chemRelatifBak;
    			}
    			//si l'entree est autre que repertoire.
    			if (entree->d_type!=4 && strcmp(entree->d_name,".")!=0 && strcmp(entree->d_name,"..")!=0){
     
    				//on allou une place mémoire pour la concaténation du chemin relatif avec le nom du fichier.
    				//Si le chemin n'est pas null on concatène le chemin relatif et le nom du fichier
    				//on lance la fonction qui va copier dans le fichier de sync, la file de fichiers.
     
    				if (chemRelatif!=NULL){
    					chemEtFichier=malloc((strlen(entree->d_name)+strlen(chemRelatif)+1)*sizeof(char));
    					strcpy(chemEtFichier,chemRelatif);
    					strcat(chemEtFichier,entree->d_name);
    				}
    				else{
    					//sinon le chemin est le nom du ficheir
    					chemEtFichier=malloc((strlen(entree->d_name)+1)*sizeof(char));
    					strcpy(chemEtFichier,entree->d_name);
    				}
    				//printf("%s",chemEtFichier);
     
    				//chemin abslue pour faire le stat!
    				cheminComplet=malloc((strlen(chemRep)+strlen(chemEtFichier)+1)*sizeof(char));
    				strcpy(cheminComplet,chemRep);
    				strcat(cheminComplet,chemEtFichier);
    				//on récupère les stats du fichier!
    				stat(cheminComplet,&statEntree);
     
    				//on enfile dans la file le chemin relatif,le nom du fichier et la date de modif
    				//printf("%u\n",*((unsigned int *)&statEntree.st_mtime));
    				enfiler(chemEtFichier,(unsigned int *)&statEntree.st_mtime,&(*fichiersPresent));
    			}
    		}
     
    	}
    	//printf("::");
     
    	//on ferme le repertoire
    	closedir(rep);
    	//on free la sauvegarde du chemin relatif
    	//free(chemRelatifBak);
    	//je sais pas comment free(cheminComplet) à voir !!
     
    }
    Help me please!
    Je ne vois pas ou j'ai déja pu free ce pointeur car d'ailleur, quand j'affiche sa valeur juste avant le free, la valeur existe toujours !!

    Merci d'avance !

  2. #2
    Membre émérite Avatar de crocodilex
    Profil pro
    Inscrit en
    Mars 2006
    Messages
    697
    Détails du profil
    Informations personnelles :
    Localisation : France

    Informations forums :
    Inscription : Mars 2006
    Messages : 697
    Par défaut
    Dans ta fonction gererSlash(), si la condition du "if" est fausse alors il n'y a pas d'allocation mémoire. Donc forcément, si tu libère une zone mémoire que tu n'as pas allouer, le free() risque de tout faire planter.

    Autre chose : il faut toujours tester ce que retourne malloc().

  3. #3
    Expert éminent
    Avatar de Emmanuel Delahaye
    Profil pro
    Retraité
    Inscrit en
    Décembre 2003
    Messages
    14 512
    Détails du profil
    Informations personnelles :
    Âge : 68
    Localisation : France, Paris (Île de France)

    Informations professionnelles :
    Activité : Retraité

    Informations forums :
    Inscription : Décembre 2003
    Messages : 14 512
    Par défaut
    Citation Envoyé par raphio000 Voir le message
    BOnjour, je suis en licence pro et je cherche une aide pour une erreur que je rencontre.
    Ton code est incomplet et ne compile pas.
    • 'chaineFichier' n'est pas défini
    • le champ 'd_type' de struct dirent n'est pas portable. (inconnu chez moi)
    • fonction 'enfiler()' inconnue
    • etc.

  4. #4
    Membre averti
    Profil pro
    Inscrit en
    Janvier 2006
    Messages
    31
    Détails du profil
    Informations personnelles :
    Localisation : France

    Informations forums :
    Inscription : Janvier 2006
    Messages : 31
    Par défaut
    Merci beaucoup crocodilex, c'est exactement ça !
    Quel newbee je fais !!
    C'était pourtant si simple....

    Résolu ....

    Pour la fonction malloc, je connaissais ce principe mais c'est vrai que j'ai tendance à oublier...

    Peux tu me donner de mauvaises experience que tu aurais rencontré à cause d'un oublie de vérification de malloc()

    Merci encore!

+ Répondre à la discussion
Cette discussion est résolue.

Discussions similaires

  1. probleme avec free.fr
    Par patsolaar dans le forum Free
    Réponses: 1
    Dernier message: 15/10/2007, 17h23
  2. [linux] probleme avec free()
    Par Asmod_D dans le forum C
    Réponses: 6
    Dernier message: 20/08/2007, 23h03
  3. Réponses: 5
    Dernier message: 05/12/2006, 14h04
  4. [MySQL] Probleme chez free avec les champs "geometry"
    Par gussoner dans le forum PHP & Base de données
    Réponses: 2
    Dernier message: 06/11/2006, 21h15
  5. [Apache] probleme htaccess free
    Par loka dans le forum Apache
    Réponses: 2
    Dernier message: 23/03/2006, 13h51

Partager

Partager
  • Envoyer la discussion sur Viadeo
  • Envoyer la discussion sur Twitter
  • Envoyer la discussion sur Google
  • Envoyer la discussion sur Facebook
  • Envoyer la discussion sur Digg
  • Envoyer la discussion sur Delicious
  • Envoyer la discussion sur MySpace
  • Envoyer la discussion sur Yahoo