grr.. encore un pb d'alloc :s (realloc & strncat)

Version imprimable

Voir 40 message(s) de cette discussion en une page

05/09/2006, 16h55
ramislebob
Citation:
Envoyé par ramislebob

Quand j'initialise mon char* à NULL, plantage :s
en revanche un :

Code:

1 2 3 char *confSrc = (char *) malloc(1); confSrc[0]=0x0;

permet une initialisation à vide.
bref, realloc semble ne s'appliquer QUE à un espace mémoire alloué par malloc.
Sous Windows, un

Code:

char *confSrc =NULL

n'est pas équivalent au malloc ci-dessus si on utilise des realloc après !
l'initialisation à NULL (ou 0x0) provoque un plantage de la CMD...

Citation:

Envoyé par Emmanuel Delahaye

Il faut vérifier si malloc() a réussi avant d'utiliser le bloc alloué.

oups... j'ai oublié de mettre de la robustesse la-dessus :s
que peut on faire quand on a un échec sur un malloc ou un realloc ? (à part retester n fois)
je repose ma question : cé koi que vous appelez cast ?
05/09/2006, 17h03
homeostasie
Citation:

Sous Windows, un

Code:

char *confSrc =NULL

n'est pas équivalent au malloc ci-dessus si on utilise des realloc après !
l'initialisation à NULL (ou 0x0) provoque un plantage de la CMD...

Si tu parles de programmation window orienté C++, le malloc est l'équivalent d'un new et le free d'un delete.

L'écriture que tu as faîte signifie que ton pointeur pointe sur une valeur non valide. C'est cela qu'il faut faire quand tu ne désires pas encore pointer sur une adresse spécifique ou si tu n'as pas encore besoin d'allouer un espace mémoire.

remarque:

Code:

char *confSrc = (char *) malloc(1);

Vaut mieux faire cela:
Code:

1 2 char *confSrc = malloc(sizeof(*confSrc));
05/09/2006, 17h25
gege2061
Citation:

Envoyé par ramislebob

Sous Windows, un

Code:

char *confSrc =NULL

n'est pas équivalent au malloc ci-dessus si on utilise des realloc après !
l'initialisation à NULL (ou 0x0) provoque un plantage de la CMD...

Ah si ! Ce genre de chose est utilisé dans les boucles, par exemple :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 char *text = NULL; size_t size = 1; while (/* ... */) { void *tmp = NULL; size <<= 2; tmp = realloc (text, sizeof (*text) * size) if (tmp) { text = tmp; } }
Si ça plante, c'est surement que tu as un débordement quelque part qui écrase les structures de données internes servant à gérer la mémoire.

Un cast c'est la même chose qu'un transtypage.
05/09/2006, 17h36
DaZumba

Citation:

Envoyé par ramislebob

que peut on faire quand on a un échec sur un malloc ou un realloc ? (à part retester n fois)

C'est une erreur fatale et on doit quitter le programme (sauf utilisation de techniques sophistiquees inutiles dans ton cas).

Citation:

je repose ma question : cé koi que vous appelez cast ?

Merci d'eviter le SMS: 'qu'appelez-vous un cast?' (ou transtypage, pour etre pedant).
C'est forcer un changement de type d'une variable, d'un retour de fonction, etc.
malloc() renvoit un void * que tu castes en (char *). En C, c'est mal, car c'est inutile (tout pointeur void est conforme a un pointeur de n'importe quel type) et cela cache la non-inclusion de stdlib.h.

Citation:

Envoyé par homeostasie

Si tu parles de programmation window orienté C++, le malloc est l'équivalent d'un new et le free d'un delete.

nan j'ai jamais fais de C++, uniquement du C.

Citation:

Envoyé par gege2061

Si ça plante, c'est surement que tu as un débordement quelque part qui écrase les structures de données internes servant à gérer la mémoire.

ben je comprends pas où alors :s
je vais remettre mon code au complet, si quelqu'un peut me dire pourquoi quand je met un NULL au lieu d'un malloc ça plante...

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
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
 
#ifdef __cplusplus
    #error Be sure you are using a C compiler...
#endif
 
#ifndef WIN32
    #define WIN32
#endif
 
#ifndef RENFOTO
    #define RENFOTO
    #include <stdio.h> 
    #include <stdlib.h>
    #include <sys/types.h>
    #include <errno.h> // perror
    #include <strings.h> // strlen, strcpy,...
    #include <string.h> // bzero...
    #include <unistd.h> // R_OK
    #include <fcntl.h> // O_RDONLY O_WRONLY
#endif
 
#ifdef WIN32
    #include <windows.h>
#endif
 
 
#define filePath "C:\\blabla.txt"
 
int existFile(char *fichier);
int myRealloc(void ** ppData, size_t newSize);
void loadTextFile(char *name, char **stockage);
 
// Fonction Principale :p
int main(int argc, char *argv[], char **envp){
    char *fichier=NULL;//(char *) malloc(1);
//    fichier[0]=0x0;
    loadTextFile(filePath, &fichier);  
    printf("%s",fichier);
    free(fichier);
    exit(0);
}
 
/** Test d'existance d'un fichier */
int existFile(char *fichier){
 
    if ( access(fichier, F_OK)==-1 ){
        printf("Error Access File : %s :\\\n",fichier);
        return -1;
    }
    else
        return 0;
}
 
int myRealloc(void ** ppData, size_t newSize){
    void *pTmp = realloc(*ppData, newSize);
    if(pTmp == NULL)
	return -1;	
    else{
	*ppData = pTmp;
	return 0;
    }
}
 
void loadTextFile(char *name, char **stockage){
    #define tailleBuffer 10
    char buffer[tailleBuffer];
    FILE *file;
    int NbLu;
    int taille=strlen(*stockage);
//    printf("TAille initiale : %i\n",taille);
 
    if (existFile(name)==-1){
	printf("Accès impossible au fichier %s...\n",name);
	printf("Existe t'il vraiment... :s");
	exit(1);
    }
 
    if((file=fopen(name, "r")) == NULL){
	printf("Ouverture impossible du fichier %s :s\n",name);	
	exit(1);
    }
 
    while( (NbLu=fread(buffer, 1, tailleBuffer, file)) != 0){
	taille+=NbLu;
	if (myRealloc((void **)stockage,taille+1)==-1){
	    free(*stockage);
	    printf("Error Realloc :s\n");
	    exit(1);
	}
//	*stockage=(char *) realloc(*stockage,taille+1);
	strncat(*stockage,buffer,NbLu);
	(*stockage)[taille-1]=0x0;
    }
    fclose(file);
}

Pourquoi ça fait planter la CMD ?

05/09/2006, 18h25
ramislebob

Citation:

Envoyé par DaZumba

C'est une erreur fatale et on doit quitter le programme (sauf utilisation de techniques sophistiquees inutiles dans ton cas).

Merci d'eviter le SMS: 'qu'appelez-vous un cast?' (ou transtypage, pour etre pedant).
C'est forcer un changement de type d'une variable, d'un retour de fonction, etc.
malloc() renvoit un void * que tu castes en (char *). En C, c'est mal, car c'est inutile (tout pointeur void est conforme a un pointeur de n'importe quel type) et cela cache la non-inclusion de stdlib.h.

sorry pour le langage SMS :s
je savais pas qu'un transtypage s'appelle un cast :s (ça me fais penser à diffusion, je vois pas trop le rapport avec un transtypage..)
05/09/2006, 18h30
Emmanuel Delahaye

Citation:

Envoyé par ramislebob

Sous Windows, un

Code:

char *confSrc =NULL

n'est pas équivalent au malloc ci-dessus si on utilise des realloc après !
l'initialisation à NULL (ou 0x0) provoque un plantage de la CMD...

Jamais constaté ça. Montre le code qui a ce comportement.

Citation:

oups... j'ai oublié de mettre de la robustesse la-dessus :s
que peut on faire quand on a un échec sur un malloc ou un realloc ? (à part retester n fois)

On peut effectivement recommencer un peu plus tard (sleep(), Sleep(), selon le système...). En cas d'echec définitif, remonter l'info aux couches supérieures qui prendront les décisions qui s'imposent. Ca dépend qui et impliqué.

Citation:

je repose ma question : cé koi que vous appelez cast ?

Je crois qu'on t'a déjà répondu : conversion d'un type dans un autre. C'est rarement utile et souvent le signe d'un codage de chacal.
05/09/2006, 18h33
Emmanuel Delahaye

Citation:

Envoyé par ramislebob

sorry pour le langage SMS :s
je savais pas qu'un transtypage s'appelle un cast :s (ça me fais penser à diffusion, je vois pas trop le rapport avec un transtypage..)

cast = rôle (casting).

Utiliser un transtypage, c'est changer le rôle d'un type.

Citation:

Envoyé par Emmanuel Delahaye

cast = rôle (casting).

Utiliser un transtypage, c'est changer le rôle d'un type.

merci pour la précision ;)

Citation:

Envoyé par Emmanuel Delahaye

Jamais constaté ça. Montre le code qui a ce comportement.

je l'ai déjà mis un peu plus haut, je le remet ici :

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
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
 
#ifdef __cplusplus
    #error Be sure you are using a C compiler...
#endif
 
#ifndef WIN32
    #define WIN32
#endif
 
#ifndef RENFOTO
    #define RENFOTO
    #include <stdio.h> 
    #include <stdlib.h>
    #include <sys/types.h>
    #include <errno.h> // perror
    #include <strings.h> // strlen, strcpy,...
    #include <string.h> // bzero...
    #include <unistd.h> // R_OK
    #include <fcntl.h> // O_RDONLY O_WRONLY
#endif
 
#ifdef WIN32
    #include <windows.h>
#endif
 
 
#define filePath "C:\\dotclear\\conf\\dotclear.ini.in"
 
int existFile(char *fichier);
int myRealloc(void ** ppData, size_t newSize);
void loadTextFile(char *name, char **stockage);
 
// Fonction Principale :p
int main(int argc, char *argv[], char **envp){
    char *fichier=NULL;//(char *) malloc(1);
//    fichier[0]=0x0;
    loadTextFile(filePath, &fichier);  
    printf("%s",fichier);
    free(fichier);
    exit(0);
}
 
 
 
 
/** Test d'existance d'un fichier */
int existFile(char *fichier){
 
    if ( access(fichier, F_OK)==-1 ){
        printf("Error Access File : %s :\\\n",fichier);
        return -1;
    }
    else
        return 0;
}
 
int myRealloc(void ** ppData, size_t newSize){
    void *pTmp = realloc(*ppData, newSize);
    if(pTmp == NULL)
	return -1;	
    else{
	*ppData = pTmp;
	return 0;
    }
}
 
void loadTextFile(char *name, char **stockage){
    #define tailleBuffer 10
    char buffer[tailleBuffer];
    FILE *file;
    int NbLu;
    int taille=strlen(*stockage);
//    printf("TAille initiale : %i\n",taille);
 
    if (existFile(name)==-1){
	printf("Accès impossible au fichier %s...\n",name);
	printf("Existe t'il vraiment... :s");
	exit(1);
    }
 
    if((file=fopen(name, "r")) == NULL){
	printf("Ouverture impossible du fichier %s :s\n",name);	
	exit(1);
    }
 
    while( (NbLu=fread(buffer, 1, tailleBuffer, file)) != 0){
	taille+=NbLu;
	if (myRealloc((void **)stockage,taille+1)==-1){
	    free(*stockage);
	    printf("Error Realloc :s\n");

Ce fichier compile, à l'exécution il plante. (seul truc à changer : ligne 26, mettre un fichier texte qui existe sur votre machine ;))

Par contre, si dans le main on remplace le NULL par le malloc (ligne 34, 35 à décommenter) il marche nickel !

Alors où est le problème si ce n'est pas le NULL ?

Citation:
Envoyé par ramislebob
uniquement du C.
je vais remettre mon code au complet, si quelqu'un peut me dire pourquoi quand je met un NULL au lieu d'un malloc ça plante...
Code:

1 2 3 4 5 6 7 8 9 10 11 12 int main(int argc, char *argv[], char **envp) { char *fichier=NULL; loadTextFile(filePath, &fichier); <...> } void loadTextFile(char *name, char **stockage) { int taille=strlen(*stockage); <...>
Pourquoi ça fait planter la CMD ?
C'est sûr que *stockage avec stockage == NULL, ça va pas le faire. On t'a déjà expliqué que la taille alloué devait être gérée à part...

06/09/2006, 12h39
ramislebob

Citation:

Envoyé par Emmanuel Delahaye

C'est sûr que *stockage avec stockage == NULL, ça va pas le faire. On t'a déjà expliqué que la taille alloué devait être gérée à part...

Oki, j'avais pas vu :s
pour la taille je la traite à part sauf ici. j'ai mis une struct comme vous le suggérez mais pour un string ce n'est pas necessaire.

Voir 40 message(s) de cette discussion en une page