c++ - 具有大字符指针的 Golang CGO - SEGSERV

Question

我有大量数据正在从 TagLib 库中读取并传递给 GoLang(mpeg 图像数据)。

这里是获取数据的地方:

void audiotags_mpeg_artwork(TagLib::MPEG::File *mpegFile, int id) {
    TagLib::ID3v2::Tag *id3v2 = mpegFile->ID3v2Tag(false);
    if (id3v2!=nullptr) {
        const TagLib::ID3v2::FrameList frameList = id3v2->frameListMap()["APIC"];
        for(auto it = frameList.begin(); it != frameList.end(); it++) {
            TagLib::ID3v2::AttachedPictureFrame * frame = (TagLib::ID3v2::AttachedPictureFrame *)(*it);
            if (frame!=nullptr && frame->size() > 0) {
                const auto &apicBase64 = frame->picture().toBase64();
                auto len = apicBase64.size();
                if (len > 0) {
                    // Generate memory for key
                    char* key = new char[5];
                    memcpy(key, "APIC", 4);
                    key[4]='\0';

                    // Generate memory for picture data
                    char* val = new char[len];
                    memcpy (val, apicBase64.data(), len);

                    // Send to GoLang
                    go_map_audiotags(id, key, val);

                    // Free memory
                    delete[] key;
                    delete[] val;
                }
            }
        }
    }
}

此时，go_map_autotags 起作用了(我对其他数据使用了类似的方法)。这也适用于其他图片数据，但是根据大小，这会导致崩溃:

unexpected fault address 0x766a000

fatal error: fault

[signal SIGSEGV: segmentation violation code=0x1 addr=0x766a000 pc=0x404530b]

在 GoLang 中，我有以下导出:

//export go_map_audiotags
func go_map_audiotags(id C.int, key *C.char, val *C.char) {
    m := maps[int(id)]
    k := strings.ToLower(C.GoString(key))
    log.Println("go_map_audiotags k:", k) // <--- works
    v := C.GoString(val) // <--- crashes
    log.Println("go_map_audiotags v:", v) // <--- Does not reach
    m[k] = v
}

有没有更好的方式来传输这些数据？我假设发生的事情是: 1) 达到 C.char 限制 2) 由于某种原因，C++ 在 Go 语言中设置 v 之前回收内存

Answer 1

val 中存储的数据不是以 null 结尾的。在您的 C 代码中，当您使用 memcpy 进行复制时，不包括空终止符。在 C 代码中，将代码更改为:

// Generate memory for picture data
char* val = new char[len+1];
memcpy (val, apicBase64.data(), len);
val[len] = '\0';