C에서 realloc()을 잘못 사용하면 어떻게됩니까?

Question

내가 물어보고 싶은 것은, 내가 메모리 관리에있어 무엇이 잘못되었는지. 난 그냥 stdin에서 일부 int 배열을 읽고 다음 그것을 인쇄 싶어요. 2 요소에서 시작하여 금액을 2로 할당하면 만족하게 될 것입니다.

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

void read (int * array, int * n, int * l) { 
    int * tmp; 
    printf("Enter values:\n"); 
    for (*l = 0; *l < *n; (*l)++) { 
     if (*l == *n - 1) { 
      (*n) *= 2; 
      tmp = (int*) realloc (array, sizeof(int) * (*n)); 
      if (tmp != NULL) { 
       array = tmp; 
       free(tmp); 
      } 
      else { 
       printf("Error!\n"); 
       free(tmp); 
     } 
     if (scanf("%d", &array[*l]) != 1) break; 
    } 
} 

void print (int * array, int length) { 
    int i; 
    printf("Your values:\n"); 
    for (i = 0; i < length; i++) printf("%d ", array[i]); 
} 

int main (void) { 
    int n = 2; /* number of array elements */ 
    int length = 0; 
    int * array = (int *) malloc(sizeof(int) * n); 

    read(array, &n, &length); 
    print(array, length); 
    free(array); 
    return 0; 
} 

Answer 1

: free(tmp);

이 문은 어디 게시 된 코드 안된다.

는 단지

==== 포인터가 'N'보다 배열을 가리키는 될 호출자, 즉 할당이 아니라, 고정 된 배열 된 할당 된 메모리를 해제

== == 포인터가 가리키는 곳에서 호출 된 함수가 변경 될 포인터를 전달할 때는 현재 시나리오에서 int **n으로 전달되어야합니다. 이것은 또한 호출하는 함수가 아닌 포인터의 내용이

==== 변수 (및 매개 변수) 이름이 '매개 변수 이름을 usage 또는 content (또는 더 나은, 둘 다)를 표시해야합니다 포인터의 주소를 통과해야 함을 의미 n '과'l '은 현재 상황에서도 의미가 없다.

Answer 2

성공적인 realloc 후 free를 제거하십시오. Realloc이 당신을 위해 그것을 처리합니다. 또한 오류 케이스에 충돌 한 경우 버퍼를 두 번 free()합니다.

Answer 3

EDIT 죄송합니다. 귀하가 복사하여 붙여 넣기 만 할 수있는 뭔가를 원했음을 깨닫지 못했습니다. 디자인 문제도 해결하겠습니다. 대한

는

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

void read (int **array, int *len) { 
    int *tmp = *array; 
    int count = *len; 
    int i = 0; 
    int val = 0; 
    char buffer[32]; 

    printf("Enter values one at a time:\n"); 
    while (i < count) { 
    if (i == (count - 1)) { 
     tmp = realloc(*array, sizeof(*tmp) * (count *= 2)); 
     if (tmp) { 
     *array = tmp; 
     } 
     else { 
     printf("Allocation failed\n"); 
     return; 
     } 
    } 
    fgets(buffer, 32, stdin); 
    if (sscanf(buffer, "%d", &val) != 1) { 
     break; 
    } 
    tmp[i++] = val; 
    } 
    *len = i; 
} 

void print (int *array, int length) { 
    int i = 0; 

    printf("Your values:\n"); 
    while (i < length) { 
    printf("%d ", array[i++]); 
    } 
} 

int main (void) { 
    int len = 2; 
    int *array = malloc(sizeof(*array) * len); 

    read(&array, &len); 
    print(array, len); 
    free(array); 
    getchar(); 
    return 0; 
}