1 回答
TA贡献1851条经验 获得超4个赞
您正在将十个(零)字节GoPack.e写入packed.ewhich 的类型为char *。这不起作用,因为指针将是 4 或 8 个字节,具体取决于您的系统,因此即使字节表示有效指针,您也会溢出分配的内存量。
如果要创建具有有效packed.e字段的有效结构,则需要在 C 堆中分配 10 个字节的内存,将字节复制到其中,然后指向packed.e此分配的内存。packed(当您释放相应的结构时,您还需要释放此内存)。你不能直接用binary.Write.
您可以以此为起点:
buf := &bytes.Buffer{}
binary.Write(buf, binary.LittleEndian, g.a)
binary.Write(buf, binary.LittleEndian, g.b)
binary.Write(buf, binary.LittleEndian, g.c)
binary.Write(buf, binary.LittleEndian, g.d)
binary.Write(buf, binary.LittleEndian, uintptr(C.CBytes(g.e))
*out = *(*C.packed)(C.CBytes(buf.Bytes()))
该函数在 C 堆中C.CBytes(b)分配len(b)字节,并将字节复制b到其中,返回一个unsafe.Pointer.
请注意,我已经*out = *(*C.packed)...从您的代码中复制了您的行。这实际上会导致内存泄漏和不必要的副本。可能最好使用将字节直接写入指向的内存的写入器out。
也许这个?
const N = 10000 // should be sizeof(*out) or larger
buf := bytes.NewBuffer((*[N]byte)(unsafe.Pointer(out))[:])
这使得 abytes.Buffer直接写入out结构而不经过任何中间内存。请注意,由于不安全的恶作剧,如果您写入的数据字节多于out.
警告:这一切都非常讨厌,并且容易出现您在 C 中发现的相同类型的问题,并且您需要检查 cgo 指针规则以确保您不会受到垃圾收集交互的影响。一点建议:鉴于您说您“对指针和内存分配没有太多经验”,您可能应该避免编写或包含这样的代码,因为它可能引入的问题是邪恶的并且可能不会立即显而易见。
- 1 回答
- 0 关注
- 371 浏览
添加回答
举报
