memcpy引起的分段错误,尽管计算了每一位

时间:2015-06-16 06:29:04

标签: c++ segmentation-fault

我正在尝试使用memcpy将数组中的数据复制到我的类的成员中。我在memcpy之前在调试器中设置了一个断点。我检查了将要使用的所有变量,并且我计算了目标中剩余的空间,看起来它应该可以工作。

#include <iostream>
#include <cstdlib>

#include <cstring>

class bigNum
{
    unsigned int dataLength;
    unsigned long long int *data;

    public:
        bigNum(){
            //long long int is 8 bytes so (2 * 1024)long long int = 16 KiB
            //  if that's not enough, we can always add more later
            dataLength = 2048;
            data = new unsigned long long [dataLength];
        };

        virtual ~bigNum(){delete[] data;};

        //bigNum& operator=(const bigNum& other);

        bigNum& set(char chars[], unsigned int charsLength) {
            //calculate where we will start writing the data
            void *writeStart = (void*)(
                (unsigned long long)data + dataLength*64 - charsLength*8
            );

            //DEBUG -- set a couple of the array elements to watch in debugger
            data[2047] = data[2046] = ~((unsigned long long)0);

            //zero out the space before writeStart
            std::memset(data, 0, dataLength*8 - charsLength);

            //write the data starting at writeStart
            std::memcpy(writeStart, chars, charsLength);

            return *this;
        }
};


using namespace std;

int main()
{
    bigNum myNum;
    char chars[9] = {'a'};

    myNum.set(chars, 9);

    system("PAUSE");
    return 0;
}

1 个答案:

答案 0 :(得分:2)

这个错误出现在这个声明中:

        void *writeStart = (void*)(
            (unsigned long long)data + dataLength*64 - charsLength*8
        );

也就是说,当您在 bytes 中执行此操作时,您似乎正在尝试在中获取正确的写入位置。

如果您有dataLength*64charsLength*8等语句,那么当您正在处理的内容时,您将在中乘以它们的大小 - 指针 - 引用字节

但是,你似乎还在玩整数尺寸。 不要这样做!这意味着您的代码将在其他机器架构上中断。不要假设unsigned long long是64位,而是确切地找出它们使用sizeof(unsigned long long)的字节数,或者如果你想要一个固定的宽度整数,请使用相应的类型,如uint64_t

然后,使用C ++,您应该坚持使用std::vector等标准容器。