16 changed files with 141 additions and 3203 deletions
--- a/.github/ISSUE_TEMPLATE/bug_report.md
+++ b/.github/ISSUE_TEMPLATE/bug_report.md
@ -1,32 +1,31 @@
 ---
-name: "[请按照此模板填写] 报告 Bug"
-about: "创建一个 Bug 报告，不按照模板的 Issue 会被直接关闭，不予回复。"
-title: "[Bug] 总结你的 Bug 报告"
+name: Bug report
+about: Create a bug report to help us improve
+title: "[Bug] Summarize the bug here"
 labels: bug
 assignees: taurusxin
+
 ---

-**Bug 描述**
+**Describe the bug**
+A clear and concise description of what the bug is.

-清晰地描述一下 Bug 的大致问题，例如无法转换，或者其他问题。
+**To Reproduce**
+Steps to reproduce the behavior:
+1. Go to '...'
+2. Click on '....'
+3. Scroll down to '....'
+4. See error

-**复现方法**
+**Expected behavior**
+A clear and concise description of what you expected to happen.

-复现此 Bug 的方法
+**Screenshots**
+If applicable, add screenshots to help explain your problem.

-1. 使用本项目处理文件 '...'
-2. 发生报错
+**Desktop (please complete the following information):**
+ - OS: [e.g. Windows 11]

-**屏幕截图**

-如果可以，屏幕截图可以更好地阐述你的问题。
-
-**环境**
-
- 操作系统: Windows / macOS / Linux
- 网易云版本（重要）: [e.g. 3.0.1]
- 所选择的音质: 极高、无损等
-
-**附加内容**
-
-如果遇到无法转换的问题，请将样本附加到这里，便于分析。
+**Additional context**
+Add any other context about the problem here.
--- a/.github/ISSUE_TEMPLATE/feature_request.md
+++ b/.github/ISSUE_TEMPLATE/feature_request.md
@ -0,0 +1,20 @@
+---
+name: Feature request
+about: Suggest an idea for this project
+title: "[Feature] Summarize new feature here"
+labels: ''
+assignees: ''
+
+---
+
+**Is your feature request related to a problem? Please describe.**
+A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
+
+**Describe the solution you'd like**
+A clear and concise description of what you want to happen.
+
+**Describe alternatives you've considered**
+A clear and concise description of any alternative solutions or features you've considered.
+
+**Additional context**
+Add any other context or screenshots about the feature request here.
--- a/.github/workflows/autobuild.yml
+++ b/.github/workflows/autobuild.yml
@ -1,6 +1,6 @@
 name: CI
 env:
-  BUILD_TYPE: MinSizeRel
+  BUILD_TYPE: Release
  BUILD_PATH: build
 on:
  push:
@ -35,7 +35,7 @@ jobs:
    - name: Upload artifact
      uses: actions/upload-artifact@v4
      with:
-        name: windows_amd64_build_msys2_exe
+        name: Windows amd64 Build - MinGW
        path: ${{ env.BUILD_PATH }}/ncmdump.exe

  build_on_windows_msvc:
@ -56,16 +56,16 @@ jobs:
    - name: Upload artifact executable
      uses: actions/upload-artifact@v4
      with:
-        name: windows_amd64_build_msvc_exe
+        name: Windows amd64 Build - MSVC
        path: ${{ env.BUILD_PATH }}/${{ env.BUILD_TYPE }}/ncmdump.exe
    
    - name: Upload artifact DLL
      uses: actions/upload-artifact@v4
      with:
-        name: windows_amd64_build_msvc_dll
+        name: Windows amd64 Build - MSVC DLL
        path: ${{ env.BUILD_PATH }}/${{ env.BUILD_TYPE }}/libncmdump.dll
  
-  build_on_linux_amd64:
+  build_on_linux:
    runs-on: ubuntu-latest

    steps:
@ -88,10 +88,10 @@ jobs:
    - name: Upload artifact
      uses: actions/upload-artifact@v4
      with:
-        name: linux_build_amd64
+        name: Linux amd64 Build
        path: ${{ env.BUILD_PATH }}/ncmdump

-  build_on_macos_amd64:
+  build_on_macos:
    runs-on: macos-latest

    steps:
@ -105,7 +105,7 @@ jobs:
        brew install git cmake

    - name: Configure build
-      run: cmake -DCMAKE_BUILD_TYPE=${{ env.BUILD_TYPE }} -B ${{ env.BUILD_PATH }} -DCMAKE_OSX_ARCHITECTURES=x86_64
+      run: cmake -DCMAKE_BUILD_TYPE=${{ env.BUILD_TYPE }} -B ${{ env.BUILD_PATH }}

    - name: Build
      run: cmake --build ${{ env.BUILD_PATH }} -j 4
@ -113,30 +113,5 @@ jobs:
    - name: Upload artifact
      uses: actions/upload-artifact@v4
      with:
-        name: macOS_build_amd64
-        path: ${{ env.BUILD_PATH }}/ncmdump
-
-  build_on_macos_arm64:
-    runs-on: macos-latest
-
-    steps:
-    - name: Checkout repository
-      uses: actions/checkout@v4
-      with:
-        submodules: recursive
-
-    - name: Install dependencies
-      run: |
-        brew install git cmake
-
-    - name: Configure build
-      run: cmake -DCMAKE_BUILD_TYPE=${{ env.BUILD_TYPE }} -B ${{ env.BUILD_PATH }} -DCMAKE_OSX_ARCHITECTURES=arm64
-
-    - name: Build
-      run: cmake --build ${{ env.BUILD_PATH }} -j 4
-
-    - name: Upload artifact
-      uses: actions/upload-artifact@v4
-      with:
-        name: macOS_build_arm64
+        name: macOS amd64 Build
        path: ${{ env.BUILD_PATH }}/ncmdump
--- a/.gitignore
+++ b/.gitignore
@ -5,5 +5,3 @@ ncmdump
 .idea

 build
-
-cmake-build-*
--- a/README.md
+++ b/README.md
@ -16,13 +16,9 @@

 ## 使用

-注意：自网易云音乐 3.0 版本开始，下载的 ncm 文件均不内置歌曲专辑的封面图片，需要从网络获取，介于在一个小工具中嵌入庞大网络库的非必要性，可以移步我的另一个仓库（<https://git.taurusxin.com/taurusxin/ncmdump-go>），下载完全使用 Golang 重写的新版本，支持从网络自动获取封面图并嵌入到目标音乐文件。
-
 ### 命令行工具

-**[不推荐]** 你可以使用 Homebrew 来安装 Linux 或者 macOS 下的 ncmdump
-
-注意：由于本项目依赖的库 taglib 的 2.0 版本尚未发布到 homebrew-core（主要是因为 taglib 2.0 导致其他 brew 包存在版本兼容问题），目前的 cmake 构建链无法在 macOS 上正常使用。根据 brew 的要求，如果依赖库已有官方的 brew 包，构建时必须使用官方仓库中的包，不能从 git 获取。而 taglib 2.0 版本开始才支持 cmake 构建。因此，ncmdump 在 homebrew 上只能发布到 `1.2.1` 版本。建议直接通过二进制方式安装，`1.3.0` 后版本修复了许多 bug，使用体验会更好。
+你可以使用 Homebrew 来安装 ncmdump 的 cli 版本

 ```shell
 brew install ncmdump
@ -36,41 +32,16 @@ brew install ncmdump
 ncmdump -h
 ```

-使用 `-v` 或 `--version` 参数来打印版本信息
+命令行下输入一个或多个文件

 ```shell
-ncmdump -v
+ncmdump file1 file2...
 ```

-处理单个或多个文件
+你可以使用 `-d` 参数来指定一个文件夹，对文件夹下的所有文件批量处理

 ```shell
-ncmdump 1.ncm 2.ncm...
-```
-
-使用 `-d` 参数来指定一个文件夹，对文件夹下的所有以 ncm 为扩展名的文件进行批量处理
-
-```shell
-ncmdump -d source_dir
-```
-
-使用 `-r` 配合 `-d` 参数来递归处理文件夹下的所有以 ncm 为扩展名的文件
-
-```shell
-ncmdump -d source_dir -r
-```
-
-使用 `-o` 参数来指定输出目录，将转换后的文件输出到指定目录，该参数支持与 `-r` 参数一起使用
-
-```shell
-# 处理单个或多个文件并输出到指定目录
-ncmdump 1.ncm 2.ncm -o output_dir
-
-# 处理文件夹下的所有以 ncm 为扩展名并输出到指定目录，不包含子文件夹
-ncmdump -d source_dir -o output_dir
-
-# 递归处理文件夹并输出到指定目录，并保留目录结构
-ncmdump -d source_dir -o output_dir -r
+ncmdump -d folder
 ```

 ### 动态库
@ -105,10 +76,6 @@ cmake -G "Visual Studio 17 2022" -A x64 -B build

 # Linux / macOS
 cmake -DCMAKE_BUILD_TYPE=Release -B build
-
-# 如果需要在 macOS 下交叉编译，可以指定 `CMAKE_OSX_ARCHITECTURES` 变量来指明目标系统架构
-cmake -DCMAKE_BUILD_TYPE=Release -DCMAKE_OSX_ARCHITECTURES=arm64 -B build  # arm64
-cmake -DCMAKE_BUILD_TYPE=Release -DCMAKE_OSX_ARCHITECTURES=x86_64 -B build  # Intel-based
 ```

 编译项目
--- a/example/csharp/NeteaseCrypt.cs
+++ b/example/csharp/NeteaseCrypt.cs
@ -14,7 +14,7 @@ namespace libncmdump_demo_cli
        private static extern IntPtr CreateNeteaseCrypt(IntPtr path);

        [DllImport(DLL_PATH, CallingConvention = CallingConvention.Cdecl)]
-        private static extern int Dump(IntPtr NeteaseCrypt, IntPtr outputPath);
+        private static extern int Dump(IntPtr NeteaseCrypt);

        [DllImport(DLL_PATH, CallingConvention = CallingConvention.Cdecl)]
        private static extern void FixMetadata(IntPtr NeteaseCrypt);
@ -42,17 +42,10 @@ namespace libncmdump_demo_cli
        /// <summary>
        /// 启动转换过程。
        /// </summary>
-        /// <param name="OutputPath">指定一个路径输出，如果为空，则输出到原路径</param>
        /// <returns>返回一个整数，指示转储过程的结果。如果成功，返回0；如果失败，返回1。</returns>
-        public int Dump(string OutputPath)
+        public int Dump()
        {
-            byte[] bytes = Encoding.UTF8.GetBytes(OutputPath);
-
-            IntPtr outputPtr = Marshal.AllocHGlobal(bytes.Length + 1);
-            Marshal.Copy(bytes, 0, outputPtr, bytes.Length);
-            Marshal.WriteByte(outputPtr, bytes.Length, 0);
-
-            return Dump(NeteaseCryptClass, outputPtr);
+            return Dump(NeteaseCryptClass);
        }

        /// <summary>
--- a/example/csharp/Program.cs
+++ b/example/csharp/Program.cs
@ -14,7 +14,7 @@ namespace libncmdump_demo_cli
            NeteaseCrypt neteaseCrypt = new NeteaseCrypt(filePath);

            // 启动转换过程
-            int result = neteaseCrypt.Dump("");  // 为空则输出到源
+            int result = neteaseCrypt.Dump();

            // 修复元数据
            neteaseCrypt.FixMetadata();
--- a/src/include/cxxopts.hpp
+++ b/src/include/cxxopts.hpp
--- a/src/include/ncmcrypt.h
+++ b/src/include/ncmcrypt.h
@ -47,7 +47,7 @@ private:
 	NcmFormat mFormat;
 	std::string mImageData;
 	std::ifstream mFile;
-	unsigned char mKeyBox[256]{};
+	unsigned char mKeyBox[256];
 	NeteaseMusicMetadata* mMetaData;

 private:
@ -66,6 +66,6 @@ public:
 	~NeteaseCrypt();

 public:
-	void Dump(std::string const&);
+	void Dump();
 	void FixMetadata();
 };
--- a/src/include/version.h
+++ b/src/include/version.h
@ -1,3 +0,0 @@
-#define VERSION_MAJOR 1
-#define VERSION_MINOR 5
-#define VERSION_PATCH 0
--- a/src/lib/libncmdump.cpp
+++ b/src/lib/libncmdump.cpp
@ -9,10 +9,10 @@ extern "C" {
        return new NeteaseCrypt(fPath.u8string());
    }

-    API int Dump(NeteaseCrypt* neteaseCrypt, const char* outputPath) {
+    API int Dump(NeteaseCrypt* neteaseCrypt) {
        try
        {
-            neteaseCrypt->Dump(outputPath);
+            neteaseCrypt->Dump();
        }
        catch (const std::invalid_argument& e)
        {
--- a/src/lib/libncmdump.h
+++ b/src/lib/libncmdump.h
@ -8,7 +8,7 @@

 extern "C" {
    API NeteaseCrypt* CreateNeteaseCrypt(const char* path);
-    API int Dump(NeteaseCrypt* neteaseCrypt, const char* outputPath);
+    API int Dump(NeteaseCrypt* neteaseCrypt);
    API void FixMetadata(NeteaseCrypt* neteaseCrypt);
    API void DestroyNeteaseCrypt(NeteaseCrypt* neteaseCrypt);
 }
--- a/src/main.cpp
+++ b/src/main.cpp
@ -10,207 +10,115 @@
 #endif

 #include "color.h"
-#include "version.h"
-#include "cxxopts.hpp"

 namespace fs = std::filesystem;

-void processFile(const fs::path &filePath, const fs::path &outputFolder)
+void displayHelp()
+{
+    std::cout << "Usage: ncmdump [-d] [-h] file1 file2 ..." << std::endl;
+    std::cout << "Options:" << std::endl;
+    std::cout << "  -d               Process files in a folder (requires folder path)" << std::endl;
+    std::cout << "  -h, --help       Display this help message" << std::endl;
+}
+
+void processFile(const fs::path &filePath)
 {
    if (fs::exists(filePath) == false)
    {
-        std::cerr << BOLDRED << "[Error] " << RESET << "file '" << filePath.u8string() << "' does not exist." << std::endl;
-        return;
-    }
-
-    // skip if not ending with ".ncm"
-    if (!filePath.has_extension() || filePath.extension().u8string() != ".ncm")
-    {
+        std::cerr << BOLDRED << "Error: " << RESET << "file '" << filePath.u8string() << "' does not exist." << std::endl;
        return;
    }

    try
    {
        NeteaseCrypt crypt(filePath.u8string());
-        crypt.Dump(outputFolder.u8string());
+        crypt.Dump();
        crypt.FixMetadata();

-        std::cout << BOLDGREEN << "[Done] " << RESET << "'" << filePath.u8string() << "' -> '" << crypt.dumpFilepath().u8string() << "'" << std::endl;
+        std::cout << BOLDGREEN << "Done: " << RESET << "'" << crypt.dumpFilepath().u8string() << "'" << std::endl;
    }
    catch (const std::invalid_argument &e)
    {
-        std::cerr << BOLDRED << "[Exception] " << RESET << RED << e.what() << RESET << " '" << filePath.u8string() << "'" << std::endl;
+        std::cerr << BOLDRED << "Exception: " << RESET << RED << e.what() << RESET << " '" << filePath.u8string() << "'" << std::endl;
    }
    catch (...)
    {
-        std::cerr << BOLDRED << "[Error] Unexpected exception while processing file: " << RESET << filePath.u8string() << std::endl;
+        std::cerr << BOLDRED << "Unexpected exception while processing file: " << RESET << filePath.u8string() << std::endl;
+    }
+}
+
+void processFilesInFolder(const fs::path &folderPath)
+{
+    for (const auto &entry : fs::directory_iterator(folderPath))
+    {
+        if (fs::is_regular_file(entry.status()))
+        {
+            processFile(entry.path());
+        }
    }
 }

 int main(int argc, char **argv)
 {
 #if defined(_WIN32)
-    win32_utf8argv(&argc, &argv); // Convert command line arguments to UTF-8 under Windows
+    win32_utf8argv(&argc, &argv);
 #endif
-
-    cxxopts::Options options("ncmdump");
-
-    options.add_options()
-    ("h,help", "Print usage")
-    ("d,directory", "Process files in a folder (requires folder path)", cxxopts::value<std::string>())
-    ("r,recursive", "Process files recursively (requires -d option)", cxxopts::value<bool>()->default_value("false"))
-    ("o,output", "Output folder (default: original file folder)", cxxopts::value<std::string>())
-    ("v,version", "Print version information", cxxopts::value<bool>()->default_value("false"))
-    ("filenames", "Input files", cxxopts::value<std::vector<std::string>>());
-
-    options.positional_help("<files>");
-
-    options.parse_positional({"filenames"});
-
-    options.allow_unrecognised_options();
-
-    // Parse options the usual way
-    cxxopts::ParseResult result;
-    try {
-        result = options.parse(argc, argv);
-    } catch(cxxopts::exceptions::parsing const& e) {
-        std::cout << options.help() << std::endl;
+    if (argc <= 1)
+    {
+        displayHelp();
        return 1;
    }

-    // print usage message if unrecognised options are present
-    if (result.unmatched().size() > 0)
-    {
-        std::cout << options.help() << std::endl;
-        return 1;
-    }
+    std::vector<fs::path> files;
+    bool processFolders = false;

-    // display help message
-    if (result.count("help"))
+    bool folderProvided = false;
+
+#define COMPARE_STR(s1, s2) (strcmp(s1, s2) == 0)
+#define HELP_SHORT "-h"
+#define HELP_LONG "--help"
+#define PROCESS_FOLDER "-d"
+    for (int i = 1; i < argc; ++i)
    {
-        std::cout << options.help() << std::endl;
+        if (COMPARE_STR(argv[i], HELP_SHORT) || COMPARE_STR(argv[i], HELP_LONG))
+        {
+            displayHelp();
            return 0;
        }
-
-    // display version information
-    if (result.count("version"))
+        else if (COMPARE_STR(argv[i], PROCESS_FOLDER))
        {
-        std::cout << "ncmdump version " << VERSION_MAJOR << "." << VERSION_MINOR << "." << VERSION_PATCH << std::endl;
-        return 0;
+            processFolders = true;
+            if (i + 1 < argc && argv[i + 1][0] != '-')
+            {
+                folderProvided = true;
+                processFilesInFolder(argv[i + 1]);
+                // Skip the folder name
+                ++i;
            }
-
-    // no input files or folder provided
-    if (result.count("directory") == 0 && result.count("filenames") == 0)
+            else
            {
-        std::cout << options.help() << std::endl;
+                std::cerr << "Error: -d option requires a folder path." << std::endl;
                return 1;
            }
-
-    // only -r option without -d option
-    if (result.count("recursive") && result.count("directory") == 0)
-    {
-        std::cerr << BOLDRED << "[Error] " << RESET << "-r option requires -d option." << std::endl;
-        return 1;
-    }
-
-    // check output folder
-    fs::path outputDir = fs::u8path("");
-    bool outputDirSpecified = result.count("output") > 0;
-
-    if (outputDirSpecified)
-    {
-        outputDir = fs::u8path(result["output"].as<std::string>());
-        if (fs::exists(outputDir))
-        {
-            if (!fs::is_directory(outputDir))
-            {
-                std::cerr << BOLDRED << "[Error] " << RESET << "'" << outputDir.u8string() << "' is not a valid directory." << std::endl;
-                return 1;
-            }
-        }
-        fs::create_directories(outputDir);
-    }
-
-    // process files in a folder
-    if (result.count("directory"))
-    {
-        fs::path sourceDir = fs::u8path(result["directory"].as<std::string>());
-        if (!fs::is_directory(sourceDir))
-        {
-            std::cerr << BOLDRED << "[Error] " << RESET << "'" << sourceDir.u8string() << "' is not a valid directory." << std::endl;
-            return 1;
-        }
-
-        bool recursive = result["recursive"].as<bool>();
-
-        if (recursive)
-        {
-            // 递归遍历源目录
-            for (const auto &entry : fs::recursive_directory_iterator(sourceDir))
-            {
-                // 没有指定输出目录，则使用源目录作为输出目录
-                if (!outputDirSpecified)
-                {
-                    outputDir = sourceDir;
-                }
-
-                // 获得递归遍历的相对路径
-                const auto &path = fs::u8path(entry.path().u8string());
-                auto relativePath = fs::relative(path, sourceDir);
-                fs::path destinationPath = outputDir / relativePath;
-
-                if (fs::is_regular_file(path))
-                {
-                    // 确保输出文件的目录存在
-                    fs::create_directories(destinationPath.parent_path());
-
-                    // 处理文件
-                    processFile(path, destinationPath.parent_path());
-                }
-            }
        }
        else
        {
-            for (const auto &entry : fs::directory_iterator(sourceDir))
+            fs::path path = fs::u8path(argv[i]);
+            files.push_back(path);
+        }
+    }
+
+    for (const auto &file : files)
    {
-                const auto &path = fs::u8path(entry.path().u8string());
-                if (entry.is_regular_file())
+        if (processFolders && fs::is_directory(file))
        {
-                    if (outputDirSpecified)
-                    {
-                        processFile(path, outputDir);
+            processFilesInFolder(file);
        }
        else
        {
-                        processFile(path, "");
+            processFile(file);
        }
    }
-            }
-        }
-        return 0;
-    }

-    // process individual files
-    if (result.count("filenames"))
-    {
-        for (const auto &filePath : result["filenames"].as<std::vector<std::string>>())
-        {
-            fs::path filePathU8 = fs::u8path(filePath);
-            if (!fs::is_regular_file(filePathU8))
-            {
-                std::cerr << BOLDRED << "[Error] " << RESET << "'" << filePathU8.u8string() << "' is not a valid file." << std::endl;
-                continue;
-            }
-
-            if (outputDirSpecified)
-            {
-                processFile(filePathU8, outputDir);
-            }
-            else
-            {
-                processFile(filePathU8, "");
-            }
-        }
-    }
    return 0;
 }
--- a/src/ncmcrypt.cpp
+++ b/src/ncmcrypt.cpp
@ -2,7 +2,6 @@
 #include "aes.h"
 #include "base64.h"
 #include "cJSON.h"
-#include "color.h"

 #define TAGLIB_STATIC
 #include "taglib/toolkit/tfile.h"
@ -16,9 +15,6 @@
 #include <string>
 #include <filesystem>

-#pragma warning(disable:4267)
-#pragma warning(disable:4244)
-
 const unsigned char NeteaseCrypt::sCoreKey[17] = {0x68, 0x7A, 0x48, 0x52, 0x41, 0x6D, 0x73, 0x6F, 0x35, 0x6B, 0x49, 0x6E, 0x62, 0x61, 0x78, 0x57, 0};
 const unsigned char NeteaseCrypt::sModifyKey[17] = {0x23, 0x31, 0x34, 0x6C, 0x6A, 0x6B, 0x5F, 0x21, 0x5C, 0x5D, 0x26, 0x30, 0x55, 0x3C, 0x27, 0x28, 0};

@ -224,7 +220,7 @@ void NeteaseCrypt::FixMetadata()
        audioFile = new TagLib::MPEG::File(mDumpFilepath.c_str());
        tag = dynamic_cast<TagLib::MPEG::File *>(audioFile)->ID3v2Tag(true);

-        if (!mImageData.empty())
+        if (mImageData.length() > 0)
        {
            TagLib::ID3v2::AttachedPictureFrame *frame = new TagLib::ID3v2::AttachedPictureFrame;

@ -239,7 +235,7 @@ void NeteaseCrypt::FixMetadata()
        audioFile = new TagLib::FLAC::File(mDumpFilepath.c_str());
        tag = audioFile->tag();

-        if (!mImageData.empty())
+        if (mImageData.length() > 0)
        {
            TagLib::FLAC::Picture *cover = new TagLib::FLAC::Picture;
            cover->setMimeType(mimeType(mImageData));
@ -257,20 +253,15 @@ void NeteaseCrypt::FixMetadata()
        tag->setAlbum(TagLib::String(mMetaData->album(), TagLib::String::UTF8));
    }

-    // tag->setComment(TagLib::String("Create by taurusxin/ncmdump.", TagLib::String::UTF8));
+    tag->setComment(TagLib::String("Create by taurusxin/ncmdump.", TagLib::String::UTF8));

    audioFile->save();
    audioFile->~File();
 }

-void NeteaseCrypt::Dump(std::string const &outputDir = "")
+void NeteaseCrypt::Dump()
 {
-    if (outputDir.empty())
-    {
    mDumpFilepath = std::filesystem::u8path(mFilepath);
-    } else {
-        mDumpFilepath = std::filesystem::u8path(outputDir) / std::filesystem::u8path(mFilepath).filename();
-    }

    std::vector<unsigned char> buffer(0x8000);

@ -301,7 +292,7 @@ void NeteaseCrypt::Dump(std::string const &outputDir = "")
                mFormat = NeteaseCrypt::FLAC;
            }

-            output.open(mDumpFilepath, std::ofstream::out | std::ofstream::binary);
+            output.open(mDumpFilepath, output.out | output.binary);
        }

        output.write((char *)buffer.data(), n);
@ -367,7 +358,7 @@ NeteaseCrypt::NeteaseCrypt(std::string const &path)

    if (n <= 0)
    {
-        std::cout << BOLDYELLOW << "[Warn] " << RESET << "'" << path << "' missing metadata infomation can't fix some infomation!" << std::endl;
+        std::cout << "[Warn] " << path << " missing metadata infomation can't fix some infomation!" << std::endl;

        mMetaData = NULL;
    }
@ -400,24 +391,23 @@ NeteaseCrypt::NeteaseCrypt(std::string const &path)
        mMetaData = new NeteaseMusicMetadata(cJSON_Parse(modifyDecryptData.c_str()));
    }

-    // skip crc32 & image version
-    if (!mFile.seekg(5, mFile.cur))
+    // skip crc32 & unuse charset
+    if (!mFile.seekg(9, mFile.cur))
    {
        throw std::invalid_argument("can't seek file");
    }

-    uint32_t cover_frame_len{0};
-    read(reinterpret_cast<char *>(&cover_frame_len), 4);
    read(reinterpret_cast<char *>(&n), sizeof(n));

    if (n > 0)
    {
-        mImageData = std::string(n, '\0');
-        read(&mImageData[0], n);
+        char *imageData = (char *)malloc(n);
+        read(imageData, n);
+
+        mImageData = std::string(imageData, n);
    }
    else
    {
-        std::cout << BOLDYELLOW << "[Warn] " << RESET << "'" << path << "' missing album can't fix album image!" << std::endl;
+        std::cout << "[Warn] " << path << " missing album can't fix album image!" << std::endl;
    }
-    mFile.seekg(cover_frame_len - n, mFile.cur);
 }
--- a/2
+++ b/2
@ -1 +1 @@
-Subproject commit e3de03501ff66221d1f1f971022b248d5b38ba06
+Subproject commit 0896fb90920c125e55248360d271d1a1674e2a4d
--- a/test/expect.bin
+++ b/test/expect.bin