Problème de realloc(): invalid next size

Bonjour,

J'ai un code qui lors de mon realloc indique l'erreur realloc(): invalid next size: 0x00000000018d8010 ***.
Pouvez-vous m'aider ?
Voici la fonction qui me donne l'erreur :

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

int make_map(char *file, t_serv *serv) { int l; int size; FILE *fd; char *line; void *test; size_t i; if ((serv->map = malloc(sizeof(char *) * 2)) == NULL || (line = malloc(sizeof(char))) == NULL) return (1); if ((fd = fopen(file, "r")) == NULL) return (2); serv->h = 0; i = 0; while (size = getline(&line, &i, fd) != -1) { serv->h++; if ((test = realloc(serv->map, sizeof(char *) * (serv->h + 2))) == NULL || (serv->map[serv->h] = malloc(sizeof(char) * (size + 1))) == NULL) return (1); l = -1; while (line[++l] != 0) serv->map[serv->h][l] = line[l]; } return (end_map(serv)); }

---

C'est la ligne en gras (19) qui me retourne l'erreur.
Merci de toute aide que vous m'apporterez.

Bonjour,

a priori, je pencherai pour un manque de mémoire, mais c'est seulement une suggestion.

Par ailleurs, pour présenter du code, il y a la balise [code], produite par le bouton # (ou [codeinline](=[c]) pour mettre dans une phrase).

Merci de votre indication pour la mise en page.

Pour le manque de mémoire cela me semble étrange vu que tout ce que je malloc dans mon code ce trouve dans cette fonction.
Et ma mémoire est sencé pouvoir supporter bien plus que ces quelques char *.

il me semble aussi qu'il faut remplacer le pointeur réalloué par la valeur de retour de realloc.

Bonjour,
La piste de Leternel me semble à creuser. Tu réalloues serv->map et du coup l'espace pointé peut ne plus être valide et pourtant tu l'utilises dans la seconde partie de ton ou. Il faut scinder ta conditionnelle :

Code:

---

1 2 3 4 5 6 7 8 9 10

if ( (test = realloc(serv->map, sizeof(char *) * (serv->h + 2)))) == NULL) gestion de l'erreur avec des free sans doute return 1; else { serv->map=test; if ( (serv->map[serv->h] = malloc(sizeof(char) * (size + 1))) == NULL) { gestion de l'erreur avec des free sans doute return 1; } ici tout est ok

---

le || était une fausse bonne idée, ca permet de ne faire le malloc que si le realloc aurait échoué. C'est le bon usage de ||, mais pas avec realloc.
Si le realloc a échoué, malloc devrait le faire aussi.

Realloc peut déplacer le pointé, en général quand il n'y a pas assez de place après lui pour simplement l'étendre.

Bonjour
Je suis d'avis avec picodev sur le contrôle des allocations.
Cependant, pourquoi faire une ré-allocation ou allocation mémoire sur un pointeur générique si(admettant allouer la mémoire de la variable "test"), celle-ci n'est pas utilisée et pourquoi realloc et à quoi cette zone mémoire servira-t-elle ?

Citation:

---

Envoyé par r49448

Code:

---

1 2	if ((test = realloc(serv->map, sizeof(char *) * (serv->h + 2))) == NULL || (serv->map[serv->h] = malloc(sizeof(char) * (size + 1))) == NULL)

---

---

Pour le bout de code qui suit, je doute qu'elle soit correcte
Pourrais-tu nous dire ce que ta fonction doit faire voir plus de détails sur la partie ci-dessous ?

Citation:

---

Envoyé par r49448

Code:

---

1 2 3 4 5 6

l = -1; while (line[++l] != 0) serv->map[serv->h][l] = line[l]; return (end_map(serv)); }

---

---

Citation:

---

Envoyé par leternel

le || était une fausse bonne idée, ca permet de ne faire le malloc que si le realloc aurait échoué. C'est le bon usage de ||, mais pas avec realloc.

---

j'en doute car le ou booléen est un peu vicieux admettant une variable X

Code:

---

[NULL == (X = ALLOC 2) || NULL == ( Y =ALLOC 2) ]

---

si X dont l'allocation est NULL répond a la condition NULL == (X = NULL), ( donc vrais ), le ou n'effectuera pas les autres testent et exécutera l'instruction prévue à cet effet ici "return (1)" et sans free des autres mémoires allouer précédemment on omettant même de fermer le fichier "fd" ouvert ( s'il est ouvert) que ça soit après traitement ou en cas d'erreur.

à bientôt

Merci à tous pour votre aide.

J'ai fini par changer d'approche car mon realloc change d'erreur au fur et à mesure que j'avançais.
J'ai donc lu mon fichier deux fois afin de savoir d'avance la taille de mon tableau.
Ce n'est certes pas très économique ni très beau.
Mais ça a l’avantage de marcher.

Merci à tous encore une fois.
Bonne journée.

tu aurais pu penser à la liste chainée, qui ne réalloue pas.

Citation:

---

Envoyé par r49448

J'ai un code qui lors de mon realloc indique l'erreur realloc(): invalid next size: 0x00000000018d8010 ***.
Pouvez-vous m'aider ?

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

int make_map(char *file, t_serv *serv) { int l; int size; FILE *fd; char *line; void *test; size_t i; if ((serv->map = malloc(sizeof(char *) * 2)) == NULL || (line = malloc(sizeof(char))) == NULL) return (1); if ((fd = fopen(file, "r")) == NULL) return (2); serv->h = 0; i = 0; while (size = getline(&line, &i, fd) != -1) { serv->h++; if ((test = realloc(serv->map, sizeof(char *) * (serv->h + 2))) == NULL || (serv->map[serv->h] = malloc(sizeof(char) * (size + 1))) == NULL) return (1); l = -1; while (line[++l] != 0) serv->map[serv->h][l] = line[l]; } return (end_map(serv)); }

---

C'est la ligne en gras (19) qui me retourne l'erreur.

---

Bonjour

Très mauvaise gestion des erreurs. Si le fichier n'est pas ouvert, on quitte la fonction avec line qui a été alloué (et pas libéré). Idem si l'allocation de line réussit mais que celle de serv->map ne réussit pas. Grosse fuite de mémoire assurée.

Ce qui amène l'allocation suivante test = realloc(serv->map, sizeof(char *) * (serv->h + 2)) dans une boucle sans que le contenu de la zone réallouée (et stocké dans la variable "test") soit utilisé. Normal que ça plante au bout d'un moment. Et t'as de la chance que ton prog plante au bon endroit parce que moi j'aurais été le processeur j'aurais fait semblant que tout marche bien et je t'aurais planté ton programme dans une fonction située à 8000km de celle-là...