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,

È stato utile?

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

Autorizzato sotto: CC-BY-SA insieme a attribuzione
Non affiliato a StackOverflow
scroll top