WinVblock 与光盘处理有关的 bug

不点

今天在 https://github.com/Sha0/winvblock/blob/master/src/winvblock/grub4dos/g4dbus.c

看到了如下的代码：

1.     /* Check for a RAM disk mapping */
   
2.     if (slot->DestDrive == 0xFF) {
   
3.         new_dev_obj = WvRamdiskCreateG4dDisk(
   
4.             slot,
   
5.             bus_dev_obj,
   
6.             media_type,
   
7.             sector_size
   
8.           );
   
9.       } else {
   
10.         new_dev_obj = WvFilediskCreateG4dDisk(
    
11.             slot,
    
12.             bus_dev_obj,
    
13.             media_type,
    
14.             sector_size
    
15.           );
    
16.       }

复制代码

在这段代码中，开发者把 0xff 号的数据盘都当成内存盘来处理，这是不对的。

当 bios 光驱（El Torito）盘号正好是 0xff 时，也会出现这种情况。

就是说，数据介质盘的盘号 0xff 存在两种可能：其一是内存介质，其二是光盘介质。

如果是内存介质，其扇区大小是正常的（即 512 字节）。如果是光盘介质，其扇区大小是 2048 字节。

以下是修复：

1.     /* Check for a RAM disk mapping */
   
2.     if (slot->DestDrive == 0xFF && ! (slot->DestODD)) {   <------- 只更动了这一句，增加了 “数据介质不是光盘” 的判断
   
3.         new_dev_obj = WvRamdiskCreateG4dDisk(
   
4.             slot,
   
5.             bus_dev_obj,
   
6.             media_type,
   
7.             sector_size
   
8.           );
   
9.       } else {
   
10.         new_dev_obj = WvFilediskCreateG4dDisk(
    
11.             slot,
    
12.             bus_dev_obj,
    
13.             media_type,
    
14.             sector_size
    
15.           );
    
16.       }

复制代码

顺便说，firadisk 的开发者没有犯这个错误。

虽然 BIOS 赋予 eltorito 光盘盘号正好是 0xff 的情况不多，并且在这种情况下创建 grub4dos 虚拟盘的情况也不多，但是，毕竟存在这种情况。
有了上述修复，代码就更完善、更健壮了。

另外，WinVblock 的以下数据结构也有小错：

1. /* From GRUB4DOS 0.4.4's stage2/shared.h */
   
2. struct S_WV_G4D_DRIVE_MAPPING_ {
   
3.     UCHAR SourceDrive;
   
4.     UCHAR DestDrive;
   
5.     UCHAR MaxHead;
   
6.     UCHAR MaxSector:6;
   
7.     UCHAR RestrictionX:1;     <---------- 此处应该有 2 位，却只有 1 位。
   
8.     UCHAR UnUsed1:1;         <---------- 这是应该新增的，补足 1 位。如果上述 RestrictionX 改成两位的话，就不要再增加 UnUsed1 了。
   
9.     UINT16 DestMaxCylinder:13; <--------- 这个位数太多了。柱面号是 10 位，因此应该改成 10。如果改成 10，下面就要增加 UnUsed2 一行了。
   
10.     UCHAR UnUsed2:3;         <---------- 这是应该新增的，补足 3 位。如果上述 DestMaxCylinder 保持 13 位的话，就不要再增加 UnUsed2 了。
    
11.     UINT16 SourceODD:1;
    
12.     UINT16 DestODD:1;
    
13.     UINT16 DestLBASupport:1;
    
14.     UCHAR DestMaxHead;
    
15.     UCHAR DestMaxSector:6;
    
16.     UCHAR RestrictionY:1;
    
17.     UCHAR InSituOption:1;
    
18.     UINT64 SectorStart;
    
19.     UINT64 SectorCount;
    
20.   };

复制代码

【补充】似乎 winvblock 本来就不支持光盘虚拟。就是说，既不支持在光盘介质上虚拟出软盘、硬盘、光盘，也不支持从软盘、硬盘介质上虚拟出光盘。我不能肯定这一点，我没有研究过，我只是猜测。

2011bigbarry

有道理。不点大师说得很对

2011bigbarry

有道理。不点大师说得很对

510819776

可惜winvblock对win7()包括32位 64位)的map方式硬盘仿真无法成功驱动,也不会修复了.

不点

初步看了 Winvblock 的代码，知道它可以处理光盘了。但处理光盘的代码是否存在 bug，还不确定。

可能最严重的问题，就是被 Win7 驱动程序签名机制卡住了。

希望有人整合 firadisk、winvblock 和 imdisk。我不懂 windows 编程，也没有开发环境，但我可以尽自己最大能力提供帮助。