在C问题中的动态内存分配

Question

我想增加a的内存，但是realloc似乎没有做任何事情。在第四个号码程序崩溃。看起来数字也被放到了[0]中，即使计数器增加了也应该是[counter]。我知道我从[1]开始，因为当我完成输入时，我正在写一个[0]的计数器。

printf的翻译：输入向量（您可以输入任何非数字字符，除了点）。

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

typedef float* vektor; 

vektor beriVektor() { 
    int counter = 0; 
    float zacasna; 
    float *a = (float *) malloc(sizeof(float)); 
    printf("Vpisi vektor (vpis zakljucis s katerim koli nestevilskim znakom razen pike):\n"); 
    while(scanf("%f", &zacasna)) { 
     counter++; 
     printf("%d\n", counter); 
     printf("%d\n", sizeof(a)); 
     a = realloc(a, (sizeof(a) + sizeof(float))); 
     a[counter] = zacasna; 
    } 
    if (sizeof(a) == sizeof(float)) { 
     return NULL; 
    } 
    a[0] = counter; 
    return a; 
} 

void izpisiVektor(vektor a) { 
    if (a == NULL) { 
     return; 
    } 
    else { 
     int velikost = sizeof(a)/sizeof(float); 
     for (int i = 0; i < velikost; i++) { 
      printf("%f", *(a+i)); 
     } 
    } 

} 

void main(){ 
    vektor a = beriVektor(); 
    izpisiVektor(a); 

} 

输出：

ragezor@ragezor-VirtualBox:~$ ./dn09.o 
Vpisi vektor (vpis zakljucis s katerim koli nestevilskim znakom razen pike): 
1 2 3 4 
1 
4 
2 
4 
3 
4 
4 
4 
*** glibc detected *** ./dn09.o: realloc(): invalid next size: 0x09052008 *** 
======= Backtrace: ========= 
/lib/libc.so.6(+0x6c501)[0x835501] 
/lib/libc.so.6(+0x71c6d)[0x83ac6d] 
/lib/libc.so.6(realloc+0xe3)[0x83af53] 
./dn09.o[0x804850e] 
./dn09.o[0x8048595] 
/lib/libc.so.6(__libc_start_main+0xe7)[0x7dfce7] 
./dn09.o[0x8048411] 
======= Memory map: ======== 
001c9000-001e3000 r-xp 00000000 08:01 393295  /lib/libgcc_s.so.1 
001e3000-001e4000 r--p 00019000 08:01 393295  /lib/libgcc_s.so.1 
001e4000-001e5000 rw-p 0001a000 08:01 393295  /lib/libgcc_s.so.1 
005d5000-005f1000 r-xp 00000000 08:01 393234  /lib/ld-2.12.1.so 
005f1000-005f2000 r--p 0001b000 08:01 393234  /lib/ld-2.12.1.so 
005f2000-005f3000 rw-p 0001c000 08:01 393234  /lib/ld-2.12.1.so 
0069e000-0069f000 r-xp 00000000 00:00 0   [vdso] 
007c9000-00920000 r-xp 00000000 08:01 393454  /lib/libc-2.12.1.so 
00920000-00922000 r--p 00157000 08:01 393454  /lib/libc-2.12.1.so 
00922000-00923000 rw-p 00159000 08:01 393454  /lib/libc-2.12.1.so 
00923000-00926000 rw-p 00000000 00:00 0 
08048000-08049000 r-xp 00000000 08:01 140607  /home/ragezor/dn09.o 
08049000-0804a000 r--p 00000000 08:01 140607  /home/ragezor/dn09.o 
0804a000-0804b000 rw-p 00001000 08:01 140607  /home/ragezor/dn09.o 
09052000-09073000 rw-p 00000000 00:00 0   [heap] 
b7700000-b7721000 rw-p 00000000 00:00 0 
b7721000-b7800000 ---p 00000000 00:00 0 
b78d9000-b78da000 rw-p 00000000 00:00 0 
b78e7000-b78eb000 rw-p 00000000 00:00 0 
bfc2e000-bfc4f000 rw-p 00000000 00:00 0   [stack] 
Aborted 

编辑： 谢谢。来自你们大家的非常好的答案。

有什么方法可以找出有多少内存空间有vektor分配？ 后来在代码中，我检查了sizeof（a）/ sizeof（float）这个数组中元素的个数，现在我知道它是不正确的。幸运的是，我有柜台存储在一个所以我知道我有多少元素，但如果我不会有这些信息存储我怎么知道？

Answer 1

见的realloc不正确。您的计算机上正在获取指针的大小，而不是所分配空间的大小。

float *a = (float *) malloc(sizeof(float)); 
/* .... */ 
a = realloc(a, (sizeof(a) + sizeof(float))); 

因此，假设一个float *占用4个字节。你会一直分配4 + sizeof(float)，最终你会走出去。你需要跟踪的元素个数，然后的：

a = realloc(a, sizeof(float) * (el_no + 1)); 

当然，一个更好的形式是：

a = realloc(a, sizeof(*a) * (el_no + 1)); 

如果以后决定改变a类型，你就可以这样安全。

编辑

作为一个侧面说明，呼吁realloc为每一个新的元素似乎是一个很好的协议，但它的速度慢。你应该采用一种策略，比如“当我用完空间时，我将把当前使用量增加一倍”，或者在这一行中。

Answer 2

a = realloc(a, (sizeof(a) + sizeof(float))); 

sizeof(a)和sizeof(float)均为常数（在大多数情况下，sizeof(anything)是常数）。所以你总是在这里要求相同数量的内存。

通常你需要沿着

a = realloc(a, counter * sizeof(float)); 

但

1. 线的东西你需要检查这对关闭的情况的一个错误：也许你会想(counter + 1)代替counter ， 或者。
2. 不要有史以来使用a = realloc(a, ...)。查看realloc如何报告错误（以及在发生错误时如何处理原始内存块）以查看原因。

也http://c-faq.com/malloc/realloc.html

Answer 3

不管你认为这个：

if (sizeof(a) == sizeof(float)) { 
     return NULL; 
} 

是干什么的，事实并非如此。

Answer 4

总之sizeof(a)不会做你认为的事情。 a的类型是*float，所以它的尺寸非常小。您需要跟踪数组的长度并将其乘以sizeof(float)以获取当前大小。

Answer 5

这样做：

float *a = (float *) malloc(sizeof(float)); 

... 
a = realloc(a, (sizeof(a) + sizeof(float))); 

的 '的sizeof（A）' 总是要（在64位机器上32位机，8）返回4。 'sizeof()'尽管它的外观不是一个功能。它是一个给你变量大小的运算符。在这种情况下，'a'只是一个指针。 sizeof不会给你指向的大小。

你需要做的是保持一个长度变量，跟踪malloc/realloc分配的块中的浮点数。

int N_floats = 0 ; 
    float *a = (float *)malloc(0) ; 
    /* add a float */ 

    a = (float *)realloc(a, (N_floats+1)*sizeof(float)) ; 
    a[N_floats] = 1.0 ; 
    N_floats += 1; 

Answer 6

您得到的错误意味着您的堆段中存在缓冲区溢出。这是因为在第15行，你做的事：

a = realloc(a, (sizeof(a) + sizeof(float))); 

的sizeof是微距功能将是威慑在编译时不运行时，你应该使用计数器来计算所需要的尺寸新的读取。尝试是这样的：

a = realloc(a, (sizeof(*a) * (counter + 1))); // + one for the counter itself 

而且你需要记住，在第21行投放至整数（计数器）和浮动，这将导致计数器来包装，并用负值重启。

输出：

gcc -m32 -O3 -Wall -Wextra -std=c99 test.c && ./a.out 
equal: true 
value = 822222233 
flt = 822222208.000000 
(int)flt = 822222208 

正如你可以看到你的计数器将腐败:)