realloc (): non valida formato next [duplicato]
Domanda
Questa domanda ha già una risposta qui:
Sto avendo un problema con la funzione realloc. Sto utilizzando solo C (in modo che nessun vettore) con libcurl. Il problema che sto avendo è che sto ottenendo il seguente errore (realloc (): non valida dimensione successiva) sul l'iterazione 12 della funzione WRITE_DATA (la funzione passo ad arricciarsi come un callback, si chiama ogni volta ha libcurl alcuni dati da restituire (dati sono passati in blocchi)).
Trace:
-Rimosso -
Fonte:
#include <stdio.h>
#include <stdlib.h>
#include <curl/curl.h>
#include <string.h>
char * Data; //stores the data
size_t RunningSize;
int write_data( char *ptr, size_t size, size_t nmemb, void *stream )
{
size_t ThisSize = (size * nmemb); //Stores the size of the data to be stored
size_t DataLen = strlen( Data ); //length of the data so far
RunningSize = (RunningSize + ThisSize ); //update running size (used as new size)
Data = realloc( Data, RunningSize ); //get new mem location (on the 12th iteration, this fails)
strcat( Data, ptr); //add data in ptr to Data
return ThisSize; //the function must return the size of the data it received so cURL knows things went ok.
}
int main( )
{
CURL *curl;
CURLcode res;
const char * UserAgent = "";
Data = malloc(1); //so realloc will work
RunningSize += 1;
curl = curl_easy_init();
if(curl)
{
curl_easy_setopt( curl, CURLOPT_NOBODY, 0 );
curl_easy_setopt( curl, CURLOPT_URL, "http://www.google.co.uk/" );
curl_easy_setopt( curl, CURLOPT_WRITEFUNCTION, write_data);
curl_easy_setopt( curl, CURLOPT_USERAGENT, UserAgent );
curl_easy_setopt( curl, CURLOPT_HEADER, 1 );
//preform request.
res = curl_easy_perform(curl);
//output the data (debugging purposes)
puts( Data );
//cleanup
curl_easy_cleanup(curl);
free(Data);
}
return 0;
}
Grazie in anticipo,
Soluzione
I dati passati alla write_data()
non è necessariamente nul-terminato; È per questo che ti dice il numero di byte.
Questo significa che non possono utilizzare strcat()
. Utilizzando è in esecuzione l'estremità della matrice e corrompere le strutture dati utilizzate da malloc
/ realloc
, quindi l'errore.
Il write_data()
dovrebbe usare memcpy()
invece, in questo modo:
int write_data( char *ptr, size_t size, size_t nmemb, void *stream )
{
size_t ThisSize = (size * nmemb); //Stores the size of the data to be stored
size_t DataLen = RunningSize; //length of the data so far
RunningSize = (RunningSize + ThisSize ); //update running size (used as new size)
Data = realloc( Data, RunningSize ); //get new mem location (on the 12th iteration, this fails)
memcpy((char *)Data + DataLen, ptr, ThisSize); //add data in ptr to Data
return ThisSize; //the function must return the size of the data it received so cURL knows things went ok.
}
Si dovrà anche RunningSize
inizializzazione a 0, non è 1. È possibile inizializzare Data
a NULL
-. Passando NULL
a realloc()
è consentito (e lo rende si comportano proprio come malloc()
)