Malloc внутри вызова функции, кажется, освобождается по возвращении?

Я думаю, что у меня это до самого основного случая:

int main(int argc, char ** argv) {
  int * arr;

  foo(arr);
  printf("car[3]=%d\n",arr[3]);
  free (arr);
  return 1;
}

void foo(int * arr) {
  arr = (int*) malloc( sizeof(int)*25 );
  arr[3] = 69;
}

Вывод:

> ./a.out 
 car[3]=-1869558540
 a.out(4100) malloc: *** error for object 0x8fe01037: Non-aligned pointer
                         being freed
 *** set a breakpoint in malloc_error_break to debug
>

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

Ответ 1

Вы передаете указатель по значению, а не по ссылке, поэтому все, что вы делаете с arr внутри foo, не будет иметь значения вне функции foo. Поскольку m_pGladiator написал один из способов, это объявить ссылку на такой указатель (возможно только в С++ btw. C не знает о ссылках):

int main(int argc, char ** argv) {
  int * arr;

  foo(arr);
  printf("car[3]=%d\n",arr[3]);
  free (arr);
  return 1;
}

void foo(int * &arr ) {
  arr = (int*) malloc( sizeof(int)*25 );
  arr[3] = 69;
}

Другой способ (лучше imho) - не передавать указатель в качестве аргумента, а возвращать указатель:

int main(int argc, char ** argv) {
  int * arr;

  arr = foo();
  printf("car[3]=%d\n",arr[3]);
  free (arr);
  return 1;
}

int * foo(void ) {
  int * arr;
  arr = (int*) malloc( sizeof(int)*25 );
  arr[3] = 69;
  return arr;
}

И вы можете передать указатель на указатель. Это C способ пройти по ссылке. Усложняет синтаксис немного, но хорошо - то, как C...

int main(int argc, char ** argv) {
  int * arr;

  foo(&arr);
  printf("car[3]=%d\n",arr[3]);
  free (arr);
  return 1;
}

void foo(int ** arr ) {
  (*arr) = (int*) malloc( sizeof(int)*25 );
  (*arr)[3] = 69;
}

Ответ 2

Вы выделили arr в foo, но это значение указателей хранится в стеке вызовов. Если вы хотите сделать это, сделайте следующее:

void foo( int ** arr) {
    *arr = (int *)malloc( sizeof(int) * 25 );
    (*arr)[3] = 69;
}

И в основном просто передайте указатель на foo (например, foo (& arr))

Ответ 3

foo получает локальную копию указателя int, alloactes память к ней и утечки этой памяти, когда она выходит за пределы области.

Один из способов исправить это, чтобы вернуть foo указатель:

int * foo() {
  return (int*) malloc( sizeof(int)*25 );
}

int main() {
    int* arr = foo();
}

Другой - передать foo указатель на указатель

void foo(int ** arr) {
   *arr = malloc(...);
}

int main() {
    foo(&arr);
}

В С++ проще изменить foo, чтобы принять ссылку на указатель. Единственное изменение, которое вам нужно в С++, - это изменить foo на

void foo(int * & arr)

Ответ 4

Так как вы передаете указатель по значению, указатель arr внутри main не указывает на выделенную память. Это означает две вещи: у вас есть утечка памяти (нет, память не освобождается после завершения функции foo), и когда вы обращаетесь к указателю arr внутри основного, вы получаете доступ к некоторому произвольному диапазону памяти, следовательно, вы не 't получить 3 распечатаны и, следовательно, free() отказывается работать. Вам повезло, что вы не получили ошибку сегментации при доступе к arr [3] внутри main.

Ответ 5

Вы не можете изменить значение своего аргумента (arr), если оно не передано по ссылке (&). В общем, вы хотели бы вернуть указатель, поэтому ваш метод должен быть:

обр = Foo();

Это плохой juju, чтобы попытаться переназначить аргументы; Я не рекомендую (&) решение.