调用 printf 更改地址 C 传递的数组

Question

我有一段代码，它通过地址（指针）获取一个数组 这个函数简单地接受一个数组和一个整数值，它是它的长度，并一个一个地打印出数组的元素。

void dumpInts(int *array, int count, int hex){
    int i;    
    for(i = 0; i < count; i++){
        printf("  %*d",10,array[i]);
    } 

上面的代码是高度简化的，下面是其余的代码。它没有那么大。

#include <stdio.h>
#include <stdlib.h>
int LAST = -1059786739;
int FIRST = -559038737;
//Input is a series of integers read in from stdin separated by whitespace
void dumpInts(int *array, int count, int hex);
void initArray(int *array, int size);

int main(){
    int bufsiz ;
    if(scanf("%d",&bufsiz) < 1){
        fprintf(stderr,"CAN\'T READ BUFSIZ\n");
        return 1;
    }
    if(bufsiz < 1){
        fprintf(stderr,"BAD BUFSIZ=%d\n",bufsiz);
        return 1;
    }
    bufsiz+=2;
    int *array = (int *)malloc((size_t)bufsiz);
    if(array == NULL){
        fprintf(stderr,"NO MORE MEM\n");
        return 1;
    }
    array[0] = FIRST;        
    array[bufsiz-1] = LAST;   
    initArray(array,bufsiz);
    dumpInts(array,bufsiz,0);
    dumpInts(array,bufsiz,1);
    return 0;
}

void initArray(int *array, int size){
    int i;
    for(i = 1; i < size - 1; i++){
        array[i] = i;
    }
}

void dumpInts(int *array, int count, int hex){
    int i;    
    for(i = 0; i < count; i++){
        printf("  %*d",10,array[i]);
} 

事情就是这样。当数组的大小小于 7 时，效果很好。从我的终端查看这个 sn-p：

-559038737 1 2 3 4 -1059786739

虽然突然间，当大小超过 7 时，最后一个元素被覆盖，随后的所有内存都设置为随机值，但第 7 个元素总是设置为相同的值...1041

下面是一个大小为 8 的数组

-559038737 1 2 3 4 5 1041 0 892677408 942878777 现在，在我走得更远之前，我觉得我应该努力，我确实花时间仔细缩小阵列被破坏的确切位置。在它进入函数之前，它是我所期望的，一旦第一个循环打印出来，数组就会被破坏。

好的，这就是它变得奇怪的地方。我通过 SSH 连接到我大学的 linux 机器，然后运行我的代码。没有破坏，数组就好了。

7 号的对比如下

我的：

-559038737 1 2 3 4 5 1041

大学：

-559038737 1 2 3 4 5 -1059786739

我要重启我的机器....

*刚刚重启，同样的问题依然存在……同样的数字……同样的大小……

另外，当我在大学计算机上编译代码并将其复制到我的机器上时，数组仍然被破坏。当我在我的机器上编译它时不是这样。

我的机器和大学机器都是基于unix的系统，运行linux。

这到底是怎么回事？！

这没有任何意义。我在内存中传递数组的地址，所以我不希望堆栈与此有任何关系？

Answer 1

您的问题中隐含的诊断调用 printf 更改按地址传递的数组 不正确：您不按地址传递数组，而只是按值传递数组元素，printf 不会改变数组。您观察到的是未定义行为的影响。

在元素数量和分配数组的大小之间存在一些混淆：您可以为数组的元素数量 (count) 使用更有意义的名称，而不是数组的大小（以字节为单位） ，传递给malloc()。

按照编码，你没有为数组分配足够的内存，初始化这个数组的代码和打印它的代码都有未定义的行为，因为你访问数组超出了它的边界。

此外，您不要使用hex 参数在十进制和十六进制输出之间进行选择。

这是一个更正的版本：

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

int LAST = -1059786739;
int FIRST = -559038737;

//Input is a series of integers read in from stdin separated by whitespace
void dumpInts(const int *array, int count, int hex);
void initArray(int *array, int size, int start);

int main(void) {
    int count;
    if (scanf("%d", &count) != 1) {
        fprintf(stderr,"Cannot read the number of elements\n");
        return 1;
    }
    if (count < 1) {
        fprintf(stderr,"Invalid count=%d\n", count);
        return 1;
    }
    count += 2;
    int *array = malloc(sizeof(*array), count);
    if (array == NULL) {
        fprintf(stderr,"Out of memory\n");
        return 1;
    }
    array[0] = FIRST;        
    initArray(array + 1, count - 2, 1);
    array[count - 1] = LAST;   
    dumpInts(array, count, 0);
    dumpInts(array, count, 1);
    return 0;
}

void initArray(int *array, int count, int start) {
    for (int i = 0; i < count; i++) {
        array[i] = start + i;
    }
}

void dumpInts(count int *array, int count, int hex) {
    for (int i = 0; i < count; i++) {
        if (hex) {
            printf("  %10x", array[i]);
        } else {
            printf("  %10d", array[i]);
        }
    }
} 

Answer 2

int *array = (int *)malloc((size_t)bufsiz);

应该是：

int *array = (int *)malloc((sizeof(int))*bufsiz);

完整解释：

void *malloc(size_t size);

是malloc的原型。如你所知 size_t 是长无符号整数。

但类型转换不是问题。

假设我输入 bufsize = 8; 然后：int *array = (int *)malloc((size_t)bufsiz);

变成int *array = (int *)malloc(10);

实际上应该是int *array = (int *)malloc(4*10);

因为我们要存储 10 个整数并考虑 sizeof(int) = 4。

所以指针指向 10 个字节而不是所需的 40 个字节。因此没有足够的内存来存储整数。

这是一个 ArrayIndexOutOfBoundsException，在 C 中不存在。

C 不做边界检查，即它从不给出这个错误。 因此，如果您继续增加指针，它只是简单地进入 malloc 的缓冲区（确实需要读/写操作）并且代码不会出现段错误，直到它用完系统分配给程序的内存。

也 你没有释放内存； free(array) 丢失了。

编辑：malloc 不计算对象计数。它占用要分配的字节数。您正在使用 malloc 作为 calloc。

普罗斯特！