ios - 内置 MemCpy Chk 将始终溢出目标缓冲区

Question

将我的应用从 32 位更新到 64 位。

根据 Apple 文档， float 只有 4 个字节，我需要使用 CGFloat(8 个字节)

我正在使用 memcpy 以字节为单位读取。我已将所有 sizeof(float) 更新为 sizeof(CGFloat)。

但是当我这样做时，我遇到了语义问题

__builtin___memcpy_chk will always overflow destination buffer. Expanded from macro memcpy

我更新了我的 NSData readDataOfLenght 以采用 sizeof(CGFloat) 并且它似乎工作正常。有时并非所有读入的数据都是正确的。

恐怕我对此不知所措，需要一些帮助。

-(void) readByteData:(NSFileHandle *)fHandle Size:(NSInteger)byteSize
{
    [super readByteData:fHandle Size:byteSize];

    NSData *data = [fHandle readDataOfLength:sizeof(CGFloat)];
    float r;
    memcpy(&r, [data bytes], sizeof(CGFloat));
    self.radius = r;

    int nCGPointSize = sizeof(CGFloat) * 2;
    data = [fHandle readDataOfLength:nCGPointSize];
    float xy[2];
    memcpy(xy, [data bytes], nCGPointSize);
    self.centerPos = ccp(xy[0], xy[1]);

    data = [fHandle readDataOfLength:sizeof(CGFloat)];
    float start_angle;
    memcpy(&start_angle, [data bytes], sizeof(CGFloat));
    self.startAngle = start_angle;

    data = [fHandle readDataOfLength:sizeof(CGFloat)];
    float end_angle;
    memcpy(&end_angle, [data bytes], sizeof(CGFloat));
    self.endAngle = end_angle;

    data = [fHandle readDataOfLength:sizeof(int)];
    int d;
    memcpy(&d, [data bytes], sizeof(int));
    self.dir = d;

    flagClosed = YES;
}

Answer 1

这条指令:

float r;
memcpy(&r, [data bytes], sizeof(CGFloat));

告诉你的编译器:

Read sizeof(CGFloat) (== 8 bytes!) from the location [data bytes] and write them to r

但是 r 只有 4 个字节! 所以前 4 个字节写入 r，接下来的 4 个字节写入内存中 r 之后的任何内容都是不允许的。 memcpy 是一个简单的字节复制指令，它将任意数量的字节从内存位置 A 移动到内存位置 B，它不能为您转换数据类型。如果您需要将 CGFloat 值转换为 float 值，那么您实际上需要自己进行转换。

CGFloat bigR;
memcpy(&bigR, [data bytes], sizeof(bigR));
self.radius = (float)bigR;

读取多个值时相同:

CGFloat bigXY[2];
data = [fHandle readDataOfLength:sizeof(bigXY)];
memcpy(bigXY, [data bytes], sizeof(bigXY));
self.centerPos = ccp((float)bigXY[0], (float)bixXY[1]);

转换只是为了让转换发生的地方更清楚，大多数编译器也会在没有所有 (float) 转换的情况下编译代码，并且不会提示。

作为一般规则:

memcpy(dst, src, size)

size 绝不能大于src指向的内存或dst指向的内存。在您的情况下，size 始终大于指向的内存 dst。

到目前为止，解释了您的代码为何不起作用。但是，您实际上根本不需要使用 memcpy 就好像您有一个内存块，其中包含已知数据类型的多个值，当然您可以直接访问该内存而无需复制它任何地方:

NSData * data = [fHandle readDataOfLength:sizeof(CGFloat)];
if (!data) {
    // ... handle errorr ...
}
const CGFloat * cgfloatsInData = (const CGFloat *)[data bytes];
self.radius = (float)cgfloatsInData[0];

data = [fHandle readDataOfLength:sizeof(CGFloat) * 2];
if (!data) {
    // ... handle errorr ...
}
const CGFloat * cgfloatsInData = (const CGFloat *)[data bytes];
self.centerPos = ccp((float)cgfloatsInData[0], (float)cgfloatsInData[1]);

等等。但这是非常低效的，因为您似乎总是期望一些没有可选值的固定大小结构，那么为什么不将其作为结构来读取呢？这样一来，您只需要一次 I/O 访问即可读取所有数据，并且系统只需创建一个 NSData 对象。

const struct {
    CGFloat radius;
    CGFloat xCoord;
    CGFloat yCoord;
    CGFloat startAngle;
    CGFloat endAngle;
    int dir;
} __attribute__((packed)) * entry;
// `const` as the memory `entry` will point to will be read-only.
// `* entry` means entry is a pointer to memory of a struct
// that looks as described above. __attribute__((packed)) means
// the memory must be laid out exactly as shown above and have no
// padding for better alignment of fields.

NSData * data = [fHandle readDataOfLength:sizeof(*entry)];
// `sizeof(*entry)` means the size of the memory entry points to,
// contrary to `sizeof(entry)` which would be the size of entry itself
// and that would simply be the size of a pointer on your system, 8 bytes,
// whereas `sizeof(*entry)` will be 44 bytes.

entry = (const void *)dataBytes;
// Any pointer type can be cased to `void *` and assigning
// a `void *` pointer so some pointer is always allowed by the compiler.

self.radius = (float)entry->radius;
self.centerPos = ccp((float)entry->xCoord, (float)entry->yCoord);
self.startAngle = (float)entry->startAngle;
self.endAngle = (float)entry->endAngle;
self.dir = entry->dir;