SDL_mixer 函数 Mix_LoadMUS_RW 导致访问冲突答案

【问题标题】：SDL_mixer function Mix_LoadMUS_RW causes Access ViolationSDL_mixer 函数 Mix_LoadMUS_RW 导致访问冲突
【发布时间】：2015-07-03 21:23:57
【问题描述】：

我在使用 SDL_mixer 从内存中加载音乐时遇到问题。以下包含一些错误检查的“最小”示例总是会因 Music::play 中的访问冲突而崩溃。

#include <SDL\SDL_mixer.h>
#include <SDL\SDL.h>
#include <vector>
#include <iostream>
#include <string>
#include <fstream>

class Music {
public:
    void play(int loops = 1);
    SDL_RWops* m_rw;
    std::vector<unsigned char> m_file;
    Mix_Music * m_music = nullptr;
};

void Music::play(int loops) {
    if (Mix_PlayMusic(m_music, loops) == -1)
        std::cout << "Error playing music " + std::string(Mix_GetError()) + " ...\n";
}

void readFileToBuffer(std::vector<unsigned char>& buffer, std::string filePath) {
    std::ifstream file(filePath, std::ios::binary);

    file.seekg(0, std::ios::end);
    int fileSize = file.tellg();
    file.seekg(0, std::ios::beg);
    fileSize -= file.tellg();

    buffer.resize(fileSize);
    file.read((char *)&(buffer[0]), fileSize);

    file.close();
}

void writeFileToBuffer(std::vector<unsigned char>& buffer, std::string filePath) {
    std::ofstream file(filePath, std::ios::out | std::ios::binary);
    for (size_t i = 0; i < buffer.size(); i++)
        file << buffer[i];
    file.close();
}

Music loadMusic(std::string filePath) {
    Music music;

    readFileToBuffer(music.m_file, filePath);
    music.m_rw = SDL_RWFromMem(&music.m_file[0], music.m_file.size());

    // Uncommenting the next block runs without problems
    /*
    writeFileToBuffer(music.m_file, filePath);
    music.m_rw = SDL_RWFromFile(filePath.c_str(), "r");
    */

    if (music.m_rw == nullptr)
        std::cout << "Error creating RW " + std::string(Mix_GetError()) + " ...\n";

    music.m_music = Mix_LoadMUSType_RW(music.m_rw, Mix_MusicType::MUS_OGG, SDL_FALSE);

    if (music.m_music == nullptr)
        std::cout << "Error creating music " + std::string(Mix_GetError()) + " ...\n";

    return music;
}

int main(int argc, char** argv) {
    SDL_Init(SDL_INIT_AUDIO);
    Mix_Init(MIX_INIT_MP3 | MIX_INIT_OGG);
    Mix_OpenAudio(MIX_DEFAULT_FREQUENCY, MIX_DEFAULT_FORMAT, MIX_DEFAULT_CHANNELS, 1024);

    Music music = loadMusic("Sound/music/XYZ.ogg");

    music.play();

    std::cin.ignore();

    return 0;
}

我的 ArchiveManager 确实可以工作，这也可以看出，因为 u 注释将缓冲区写入文件的块并从中创建 SDL_RW 将运行得很好。我加载的音乐文件只是假设是一个 ogg 文件，在这种情况下就是这样，因此从文件创建一个 SDL_RW 工作正常。这意味着没有任何崩溃，音乐从头到尾正常播放。

据我所知，音乐课太大了。我只是保留缓冲区 m_file 以及 SDL_RW 以确保问题不是来自被释放的数据。使用 SDL_FALSE 运行 Mix_LoadMUS_RW 还应确保未释放 RW。

值得注意的是，使用 Mix_LoadWAV_RW 从同一存档中加载 wav 文件的类似示例可以正常工作：

Mix_Chunk * chunk;
std::vector<unsigned char> fileBuf = ArchiveManager::loadFileFromArchive(filePath);
chunk = Mix_LoadWAV_RW(SDL_RWFromConstMem(&fileBuf[0], fileBuf.size()), SDL_TRUE);

在调用 Mix_PlayCannel 之前，我什至没有保留缓冲区。同样在这里，我使用 SDL_TRUE 调用加载函数，因为我没有创建显式 SDL_RW。尝试类似的方法来加载音乐不会有什么不同。

我研究了 SDL_mixer 源代码，但对我没有帮助。也许我的知识不够，或者我错过了一些重要的东西。

直截了当：访问违规从何而来，我该如何预防？

编辑：更改了示例代码，因此任何人都可以直接复制它。所以没有 ArchiveManager 或类似的东西，只是将 ogg 直接读入内存。关键部分只是 loadMusic 中的几行。

【问题讨论】：

标签： c++ sdl-2 sdl-mixer

【解决方案1】：

Music music = loadMusic("Sound/music/XYZ.ogg");
music.play();

第一行将右侧的class Music类型的对象复制到名为music的新对象中。这将导致向量 m_file 被复制，包括其中的数据。我们的新对象 music 的向量数据显然将存储在与 loadMusic 返回的对象向量不同的内存位置。然后 loadMusic 返回的对象将从堆栈中删除，其向量的数据将被释放，从而使先前创建的 Mix_Music 对象无效并导致第二个访问冲突行。

这可以通过只创建一个 Music 对象来解决，例如通过在堆上通过 new 创建它并让 loadMusic 返回指向该对象的指针。

Music* music = loadMusic("Sound/music/XYZ.ogg");
music->play();

无论如何，在堆上而不是在堆栈上为整个文件分配内存可能是更好的选择，尽管我猜向量会在内部执行此操作。

如此简短的版本，这是（我认为）一个新手错误，我太专注于责怪 SDL_Mixer。坏主意。

【讨论】：