首页 > 解决方案 > 从 memcpy 打印结果的问题

问题描述

我试图用来memcpy挑选我感兴趣的特定范围的数据,比如使用来自Matlab array(100:200). 但是,当我尝试打印结果以检查该功能是否正确时,我收到了一个错误:Exception thrown at 0x00007FFF4A921399 (vcruntime140d.dll) in Examplefordebug.exe: 0xC0000005: Access violation writing location 0x0000000000000000.有解决这个问题的想法吗?结果假设是5数组中的两个ch2Buffer

这是示例代码:

#include <iostream>
#include <stdio.h>
#include <string.h>

int main()
{
    int i,n;
    const int u32Size = 10;
    float* ch1Buffer = NULL;
    double* ch2Buffer = NULL;
    double* ch2newBuffer = NULL;
    
    int pBuffer[u32Size] = {10,2,10,2,10,5,10,5,10,2};
    int* pi16Buffer = pBuffer;

    ch1Buffer = (float*)calloc(u32Size, sizeof* ch1Buffer);
    ch2Buffer = (double*)calloc(u32Size, sizeof* ch2Buffer);
    

    // De-interveal the data to ch1Buffer and ch2Buffer
    for (i = 0; i < u32Size/2; i++)
    {
        ch1Buffer[i] += pi16Buffer[i * 2];
        ch2Buffer[i] += pi16Buffer[i * 2 + 1];
    }

    // Use memcpy to pick out the data we are interested
    memcpy(ch2newBuffer, &ch2Buffer[2], 2 * sizeof(ch2Buffer[0]));



    // Print out to check the result
    for (i = 0; i < 3; i++) {
        printf("%f ", ch2newBuffer[i]);
    }

    free(ch1Buffer);
    free(ch2Buffer);
    return 0;
}

标签: arrayscsortingprintfmemcpy

解决方案


“访问冲突写入位置 0x0000000000000000。有解决这个问题的想法吗?”

是的,您的代码正在尝试写入该进程尚未拥有的内存位置...
您在此处有一个声明:

double* ch2newBuffer = NULL;

然后尝试将对象复制到它:

memcpy(ch2newBuffer, &ch2Buffer[2], 2 * sizeof(ch2Buffer[0]));

无需先分配内存。分配内存将解决问题。

添加第三个内存分配,并在尝试使用缓冲区之前检查每个结果。一些额外的建议,不要在 C 中转换 (m)(c)(re)alloc() 的返回,并且不要在[m][c][re]alloc()没有首先验证调用是否成功的情况下使用创建的内存。这是包含这些建议的重构部分:

ch1Buffer = calloc(u32Size, sizeof* ch1Buffer);
if(ch1Buffer)
{
    ch2Buffer = calloc(u32Size, sizeof* ch2Buffer);
    if(ch2Buffer)
    {
        ch2newBuffer = calloc(u32Size, sizeof* ch2Buffer);
        if(ch2newBuffer)
        {
            // De-interveal the data to ch1Buffer and ch2Buffer
            for (i = 0; i < u32Size/2; i++)
            {
                ch1Buffer[i] += pi16Buffer[i * 2];
                ch2Buffer[i] += pi16Buffer[i * 2 + 1];
            }

            // Use memcpy to pick out the data we are interested
            memcpy(ch2newBuffer, &ch2Buffer[2], 2 * sizeof(ch2Buffer[0]));



            // Print out to check the result
            for (i = 0; i < 3; i++) {
                printf("%f ", ch2newBuffer[i]);
            }
            free(ch2newBuffer); 
        }else {/*handle error*/}
        free(ch2Buffer);
    }else {/*handle error*/}
    free(ch1Buffer);
}else {/*handle error*/}

一个额外的潜在问题...根据您的编译器,您可能在此处注册了一个额外的错误:

 const int u32Size = 10;
 ...
 int pBuffer[u32Size] = {10,2,10,2,10,5,10,5,10,2};

可变大小的对象VLA(在 C99 中定义,从那时起在编译器中定义。)可能不会在声明时初始化。(在此处阅读有关此内容的更多信息

对于具有初始化程序的非 VLA 数组,可以按以下方式进行声明,该数组可以有多种形式:

#define U32_SIZE 10

 int pBuffer[10] = {10,2,10,2,10,5,10,5,10,2};
 int pBuffer[U32_SIZE] = {10,2,10,2,10,5,10,5,10,2};
 int pBuffer[] = {10,2,10,2,10,5,10,5,10,2};

或者 VLA 的以下内容(无法初始化,只能在自动范围内声明。)

 int iBuffer[] = {10,2,10,2,10,5,10,5,10,2};//non-VLA for illustration
 int pBuffer[u32Size];//VLA 
 memcpy(pBuffer, iBuffer, sizeof(iBuffer);//update VLA with contents of iBuffer.
 

推荐阅读