realloc sous code block

Version imprimable

Bonjour,

J'ai un petit problème avec un bout de code:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
#include <stdio.h>
#include <stdlib.h>
 
int main()
{
 
char *buffer=(char*)malloc(1);
int tab=0;//taille tableau = tab+1
 
                    while(1)
                    {
                        scanf("%c",&buffer[tab]);
                        printf("%c",buffer[tab]);
                        buffer=(char*)realloc(buffer,(tab+1));
                        if(buffer==NULL)
                        {
                            perror("realloc");
                            exit(-1);
                        }
                        tab++;
                    }
    return 0;
}

je le compile(sous codeblock) et je l'exécute.
Lorsque je met par exemple, "pppppppppppp" le programme me retourne:
"ppppppppprealloc: Not enough space". grace à ma fonction perror.

Et bizarrement si le compile sous linux aucun problème.
D'où pourrait venir le problème?

merci d'avance

28/03/2010, 15h53
Obsidian

Attention : tu fais ton realloc(tab+1) avant d'avoir incrémenté tab. Ton realloc() alloue donc une taille identique la première fois (1 octet), et le caractère suivant est enregistré en dehors du tableau.
28/03/2010, 16h11
sebatlante

oh le ***... lol
en effet incrémenter avant fonctionne mieux:D
par contre c'est quand même bizarre que ça fonctionnait quand même sous linux. ça aurait aussi du bugger non?
en tous cas je te remercie Obsidian.
28/03/2010, 16h33
Obsidian

Citation:

Envoyé par sebatlante

oh le ***... lol
en effet incrémenter avant fonctionne mieux:D
par contre c'est quand même bizarre que ça fonctionnait quand même sous linux. ça aurait aussi du bugger non?

Pas forcément : la mémoire est allouée par blocs. Tant que tu restes dans un segment appartenant à ton processus, le processeur considère que ce n'est pas ses oignons. Par contre, tu risques d'aller écraser des données adjacentes et là, ça devient difficile de retrouver le bug.

Citation:

en tous cas je te remercie Obsidian.

À ton service ! ;)
28/03/2010, 17h22
jeroman

Attention, si ton realloc échoue, il retourne NULL, et la mémoire allouée précédemment par malloc n'est ni libérée ni libérable, vu que tu écrases la valeur du pointeur. Cela provoque donc une fuite de mémoire.

Il faut stocker la valeur retour dans un pointeur temporaire. Si cette valeur n'est pas NULL, tu peux écraser la valeur de ton pointeur par celle du pointeur temporaire. Sinon, il faut gérer l'erreur. Par exemple, je ne suis personnellement pas très ami avec la fonction exit :
A ) Elle casse la structure d'un programme. En principe, si une erreur se produit dans une fonction, elle doit la gérer elle-même ou le faire savoir à la fonction appelante (via la valeur retour par exemple), qui doit elle-même la gérer ou à nouveau la faire gérer par la fonction qui l'a elle-même appelée, etc.
B ) L'utilisateur voit le programme se terminer de manière inattendue, il ne comprend pas ce qui lui arrive.
C ) En principe, un programme qui alloue de la mémoire doit lui-même la nettoyer une fois qu'il n'en a plus besoin. Si on termine un programme "sauvagement", c'est l'O.S. qui est alors obligé de s'en charger. C'est pas très propre.

:ccool: