Как выделить память в другой функции, передавая pointer через аргументы?

Question

[bqio]

#include <stdio.h>
#include <stdlib.h>

#define TEST_STR "123"
#define TEST_STR_SIZE 3

static void update_callback(char *ptr)
{
    ptr = (char *)malloc(sizeof(char) * TEST_STR_SIZE + 1);
    ptr = TEST_STR;
    ptr[TEST_STR_SIZE] = 0;
}

int main(void) {
    char *nilstr = NULL;
    
    update_callback(nilstr);
    
    printf("%s\n", nilstr); // Segmentation fault
    
    return 0;
}

Answer 1

[Wataru]

В вашем случае логичнее возвращать указатель на вделенную память.

Если уж очень надо через аргументы, то передавайте указатель на указатель:

static void update_callback(char **ptr)
{
    *ptr = (char *)malloc(sizeof(char) * TEST_STR_SIZE + 1);
...

P.s. В вашем коде утечка памяти. Вы выделяете память, потом единственный указатель на эту память переписываете указателем на TEST_STR.

Comments

[bqio]

А можете подробнее рассказать об утечке, потому что не могу понять, где она. Я разве не выделил malloc 3 байта (не считая null-terminated) и потом поместил в них тоже три байта, TEST_STR разве не три байта занимает?

[Wataru]

bqio, Вы выделили память, malloc, условно говоря, вернул вам адресс 0x1234000. Потом вы в переменную ptr записали TEST_STR, которая является указателем на константную строку, лежащую по адресу, допустим, 0x00FF12CE.
Теперь ptr указывает на 0x00FF12CE, а выделенная память по адресу 0x1234000 потеряна навсегда.

Если вы хотите записать в выделенную память строку, то вам надо воспользоваться strcpy или memcpy.

Ну и учитите, что длина вашей строки - 4 байта: 3 циферки и терминирующий байт '\0' на конце.

[bqio]

Wataru, спасибо большое.

[bqio]

Wataru, а не подскажите, что на этот раз я не так сделал? Почему у меня в buf в итоге в конце ничего нет?

#include <stdio.h>
#include <stdlib.h>
#include <curl/curl.h>

size_t write_callback(void *data, size_t size, size_t nmemb, void *userp)
{
  size_t realsize = size * nmemb;
  memcpy(userp, data, realsize);
  return realsize;
}

int main(void)
{
  CURL *curl;
  CURLcode res;
  char *buf;

  curl_global_init(CURL_GLOBAL_ALL);
  curl = curl_easy_init();

  if (curl)
  {
    curl_easy_setopt(curl, CURLOPT_URL, "https://curl.se/");
    curl_easy_setopt(curl, CURLOPT_WRITEFUNCTION, write_callback);
    curl_easy_setopt(curl, CURLOPT_WRITEDATA, (void *)buf);

    res = curl_easy_perform(curl);

    if (res != CURLE_OK)
    {
      fprintf(stderr, "curl_easy_perform() failed: %s\n", curl_easy_strerror(res));
    }

    printf("%s", buf);

    curl_easy_cleanup(curl);
  }

  curl_global_cleanup();

  return EXIT_SUCCESS;
}

[Wataru]

bqio, Вы память под buf не выделяете. Это указатель на случайную память. Может ваш колбак и пишет туда данные, но потом настоящий хозяин этой памяти там все переписывает.
Проверьте еще, а вызывается ли у вас write_callback вообще. Вставьте туда printf какой-нибудь для отладки.

Потом, я не уверен, что вообще эта программа делать должна и как работает curl, но может быть данные, полученные такми методом, нельзя выводить как строку. Может они с нулевого байта начинаются и printf считает. что получил пустую строку.

[bqio]

Wataru, ну вот здесь пример со структурой, а мне не хочется так усложнять, а хочется просто в строку сразу ответ вернуть.

https://curl.se/libcurl/c/getinmemory.html