如何阻止函数内存泄漏

Question

TL；dr:如何在没有随机文本的情况下将无符号32位整数转换为chars/uint8_t

好的，我愿意为此牺牲几个声誉点。我需要快速将一个4字节的无符号整数转换为数组字节，以便读取/写入/操作我自己结构的二进制文件。
这样我就可以读取一个结构，然后将其用作对象，而不是读取它，为每个更改写入它。

但是当我尝试实现一个函数时，我得到了一个泄漏。指针只是不断在函数范围之外添加值。

void trans_byte( uint32_t &A, uint8_t *out ){
    // i did this with loop too same thing happens
    uint8_t *ptr = (uint8_t*)&A;
    out[3] = *ptr;
    ptr++;
    out[2] = *ptr;
    ptr++;
    out[1] = *ptr;
    ptr++;
    out[0] = *ptr;
    ptr = ptr - 4;
}

uint32_t trans_ray( uint8_t *in ){
    return ( in[0] << 24 ) | ( in[1] << 16 ) | ( in[2] << 8 ) | in[3]; 
}

主要：

int main(){
    std::cout << "SHUGA" << std::endl; // test printing to see if test works 

    uint32_t ADR = 0x6D4E4344; // memory should write magic of mNCD

    uint8_t ret[4];// array to be "outed" from function
    trans_byte(ADR, ret); // function
    uint8_t ret2[4] = { ret[0], ret[1], ret[2], ret[3] }; //copy of array to see if issue happens if i copy it

    std::cout << ret << std::endl; // printing the array 1
    std::cout << ret2 << std::endl; // printing the array 2
}

和输出:

SHUGA
mNCDmNCD // keep in mind I called the function only once. 
mNCD

在我添加到main之后

std::cout << "shiya" << std::endl;
uint32_t ADR2 = trans_ray(ret);

std::cout << ret << std::endl;
std::cout << ret2 << std::endl;
std::cout << (ret==ret2) << std::endl;

然后我得到了这个可爱的混乱：

SHUGA
mNCDmNCD─[uÉ@
mNCD─[uÉ@
shiya
mNCDmNCD─[uÉ@
mNCD─[uÉ@
0

所以我猜这是某种内存泄漏，指针会继续读取内存或稍后的内容（因此会有过多的符号）。

那么，我怎样才能从uint32_t到uint8_t来回转换，而不用指针继续读取和操作数组，即使它没有被调用呢？

在您开始注释之前:< br >我不想使用< code>std::vector,因为我不想在现有的大型项目(无论是file还是PE)中为每个文件添加20kb左右的二进制数据。< br >我不能使用< code>std::string，因为我不想寻找字符串结尾< code>\x00。< br >我尝试使用< code>unions，但是字节是颠倒的，对于幻数和类似的东西，我需要它们按顺序排列。即使我尝试反转数组，我仍然需要处理这个问题。< br >我尝试使用< code>struct-ures和< code>class-es，但是当我更改object中的某些内容并需要更新文件时，我在编写代码块时仍然会遇到这个问题。

Answer 1

如何将无符号的 32 位整数转换为没有随机文本的 chars/uint8_t

您可以< code>union来实现这一点，但是您必须了解字节序。试试这个:

#include <stdint.h>

union {
    uint32_t _32ui;
    uint8_t _8ui[4];
}

我尝试使用联合，但字节是颠倒的，对于幻数和类似的数字，我需要它们保持有序。即使我尝试反转数组，我仍然需要解决这个问题。

在big-endian或litt-endian顺序中有转换uint32_t的函数。我不确定您的操作系统、处理器和编译器。但是有像htonl这样的函数可用于Linux。

所以我猜这是某种内存泄漏，指针会继续读取内存或稍后的内容（因此会有过多的符号）。

不，这不是问题所在。您没有在程序中分配任何会导致内存泄漏的内存，也没有超过分配的内存缓冲区。问题是您没有通过std::c留档，您可能忽略了编译器的警告。

< code>std::cout已为C样式字符串重载，它需要< code >空终止字符数组。但是你没有提供空字符数组。我建议写一个简单的函数来打印你的数组。

void printArray(uint8_t *array, size_t size)
{
    for (size_t i = 0; i < size; i++)
        std::cout << array[i];
    std::cout << std::endl;
}

Answer 2

 std::cout << ret

这需要一个空终止的字符串（此处的信息）：

8） 输出一个实现定义的字符串，就好像通过 *this

否则，它将继续打印，直到找到\0。

这就是为什么我们使用向量，字符串等。忘记所有这些C风格的80年代的程序。除非你正在用20KB RAM编写超级微控制器，否则内存应该不是问题。