Discussion :

[ramislebob]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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...

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 ?

[homeostasie]

Sous Windows, un

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

Vaut mieux faire cela:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[gege2061]

Sous Windows, un

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[DaZumba]

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).

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.

[ramislebob]

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.

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 :

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
 
#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 ?

[ramislebob]

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..)

[Emmanuel Delahaye]

Sous Windows, un

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

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é.

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.

[Emmanuel Delahaye]

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.

[ramislebob]

cast = rôle (casting).

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

merci pour la précision

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 :

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
 
#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 ?

[Emmanuel Delahaye]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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...

[ramislebob]

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.