Проблемы с использованием realloc в C

Есть две разные проблемы.

Во-первых, линия

nptr = (int *)realloc(data_aux,(br+br_aux));

делает три вещи:

1. Он пытается выделить br + br_aux байта.
2. Он может освободить память, на которую указывает data_aux.
3. Он указывает nptr на адрес вновь выделенной памяти.

Но код продолжает использовать data_aux, как будто он все еще указывает на новую память.

Во-вторых, поскольку recv() возвращает количество полученных байтов, вы должны использовать эту информацию для добавления данных в буфер:

while (br > 0) {
  memcpy(data_aux + br_aux, data, br);
  br_aux += br;
  br = recv(sockdata, data, size_data, 0);
  if (br > 0) {
    nptr = (int *) realloc(data_aux, br + br_aux);
    if (nptr == NULL) {
      // ERROR
    }
    data_aux = nptr;
  }
}

Проблема с recv() в сочетании с любой из строковых операций (например, strcat()) заключается в том, что recv() может возвращать двоичные данные. Если эти данные содержат нулевой байт, строковые функции будут считать, что это конец данных, и будут вести себя так, как вы не ожидаете и не хотите.

На самом деле существует третья проблема: ни одно из возвращаемых значений не проверяется на наличие ошибок. Всегда проверять наличие ошибок, вместо того, чтобы предполагать, что память действительна или что связь через socked прошла успешно.

В коде много проблем, но я решу самые серьезные:

1. Вы выделяете память и предполагаете, что она готова к использованию:

   data_aux = malloc(br);
   ...
   strcat(data_aux, data); /* <- who said data_aux isn't garbage? */
   

2. # P3 #

   # P4 #

   data_aux = realloc(data_aux, br + br_aux); /* reassign data_aux */
   

3. Вы не проверяете никаких возвращаемых значений. Большой не проверяет результат recv() перед выделением ему памяти:

   br = recv(...);
   ...
   data_aux = malloc(br); /* -1 as a size is large in unsigned speak */
   

4. Вы используете строковые функции ASCIIZ для данных, которые могут даже не содержать chars. Используйте memcpy вместо strcat или strncat.

Эээ - строка, которую вы получаете из сокета, завершается нулем?

Предложения:

1. Убедитесь, что вы включили нулевой байт в свое сообщение, прежде чем отправлять его.

2. Используйте strncat ()

3. Обнулить буфер самостоятельно, если необходимо

PS: Я предполагаю, что вы переполняете буфер в вашем «strcat (data_aux)», что непреднамеренно повреждает как «data», так и «data_aux».

Если realloc() завершается успешно, входной блок памяти больше не действителен, и возвращаемый указатель указывает на новый блок памяти. Если realloc() не удается, блок входной памяти все еще действителен. Вы назначаете возвращаемый указатель переменной nptr, которая вообще ни для чего не используется, и никогда не обновляете переменную data_aux, чтобы она указывала на перераспределенную память. Когда вы вызываете free(), чтобы освободить data_aux, если realloc() был вызван заранее и был успешным, вы освободите неправильный указатель, что может привести к сбоям, как вы видите.

Измените это:

nptr = (int *) realloc(data_aux,(br+br_aux)); 

Вместо этого:

nptr = (char*)realloc(data_aux, br_aux + br); 
if (!nptr)
{
    ... error handling ...
    break;
}
data_aux = nptr;

С учетом сказанного, вам следует переписать всю логику, чтобы вместо этого было что-то вроде следующего. Нет необходимости вызывать recv() несколько раз в цикле:

int size_data = 1024 * sizeof(char);                
char *data_aux = NULL;
int br_aux = 0;  
char *nptr;  

char *data = malloc(size_data);  
if (data)
{
    int br = recv(sockdata, data, size_data, 0);
    while (br > 0)
    {  
        nptr = (char*) realloc(data_aux, br_aux + br);
        if (!nptr)
            break;

        data_aux = nptr;
        memcpy(&data_aux[br_aux], data, br);  
        br_aux = br_aux + br;  
    }  

    free(data);  
}

printf("%.*s", br_aux, data_aux);  
free(data_aux);  

Для дальнейшего упрощения поместите вместо этого буфер data в стек:

char* data_aux = NULL;
int br_aux = 0;  

char data[1024];
char *nptr;  

int br = recv(sockdata, data, sizeof(data), 0);
while (br > 0)
{  
    nptr = (char*) realloc(data_aux, br_aux + br);
    if (!nptr)
        break;

    data_aux = nptr;
    memcpy(&data_aux[br_aux], data, br);  
    br_aux = br_aux + br;  
}  

printf("%.*s", br_aux, data_aux);  
free(data_aux);