--- nono/vm/msxdos.cpp 2026/04/29 17:05:17 1.1 +++ nono/vm/msxdos.cpp 2026/04/29 17:05:25 1.1.1.3 @@ -149,61 +149,51 @@ MSXDOSDevice::Init() xs.WriteString("trap occured!\x0d\x0a$"); // 実行ファイルオープン - const char *exec_file = gMainApp.exec_file; - data_t src = LoadBinary(exec_file); - if (src.data == NULL) { + std::vector src; + if (LoadBinary(gMainApp.exec_file, src) == false) { return false; } // コピー - if (src.size >= 0x7000 - 0x100) { - src.size = 0x7000 - 0x100; - } xs.SetAddr(0x100); - for (int i = 0; i < src.size; i++) { - xs.Write8(src.data[i]); + for (auto s : src) { + xs.Write8(s); } - delete[] src.data; - return true; } -// filename を data_t にロードする。 -// 成功すれば data_t を埋めて返す。 -// その場合 data_t.data は new[] してあるので呼び出し側で delete[] すること。 -// 失敗すればエラーメッセージを表示し、data_t.data = NULL を返す。 +// filename で示されるファイルをロードする。 +// 成功すれば data に格納して true を返す。 +// 失敗すればエラーメッセージを表示し、false を返す。 // サポートしているのは // o z80-asm の出力する .z80 形式 // o MSX-DOS の .COM 形式 (生バイナリ) -MSXDOSDevice::data_t -MSXDOSDevice::LoadBinary(const char *filename) +bool +MSXDOSDevice::LoadBinary(const char *filename, std::vector& dst) { struct stat st; - data_t dl; + size_t dlsize; char header[10]; autofd fd; int r; - dl.data = NULL; - dl.size = 0; - fd = open(filename, O_RDONLY); if (fd < 0) { warn("\"%s\" open failed", filename); - return dl; + return false; } r = fstat(fd, &st); if (r < 0) { warn("\"%s\" fstat failed", filename); - return dl; + return false; } r = read(fd, header, sizeof(header)); if (r < 0) { warn("\"%s\" read(header) failed", filename); - return dl; + return false; } if (r < sizeof(header)) { warnx("\"%s\" read(header): %d: too short", filename, r); @@ -211,41 +201,35 @@ MSXDOSDevice::LoadBinary(const char *fil if (strncmp(header, "Z80ASM\x1a\x0a", 8) == 0) { // .z80 形式。先頭 10 バイトがヘッダ。 - dl.size = st.st_size - 10; + dlsize = st.st_size - 10; } else { // そうでなければ今の所 .COM 形式。ヘッダなしの生バイナリ。 - dl.size = st.st_size; + dlsize = st.st_size; if (lseek(fd, 0, SEEK_SET) < 0) { warn("\"%s\" lseek(0) failed", filename); - return dl; + return false; } } - try { - dl.data = new uint8 [dl.size]; - } catch (...) { - dl.data = NULL; - warnx("Could not allocate memory %d", dl.size); - return dl; + // サイズ制限 + if (dlsize >= 0x7000 - 0x100) { + errno = EFBIG; + warn("%s", filename); + return false; } - r = read(fd, dl.data, dl.size); + dst.resize(dlsize); + r = read(fd, dst.data(), dst.size()); if (r < 0) { warn("\%s\" read failed", filename); - goto abort; + return false; } - if (r < dl.size) { + if (r < dlsize) { warn("\%s\" read: %d: too short", filename, r); - goto abort; + return false; } - return dl; - - abort: - delete[] dl.data; - dl.data = NULL; - dl.size = 0; - return dl; + return true; } // コールバック入口のグローバル関数 @@ -263,7 +247,7 @@ MSXDOSDevice::Syscall() switch (xp->reg.c) { case 0x00: // _TERM putmsg(1, "DOS _TERM"); - Human68k::RequestExit(0); + Human68kDevice::RequestExit(0); break; case 0x02: // _PUTC