Re: [PATCH v5 0/6] read-only UDF file system support

[Paulo Alcantara <pcacjr@zytor.com>]

Ray,

On 07/09/2017 20:13, Paulo Alcantara wrote:
> v5:
>   - Fixed OVMF IA32 build.
>   - Fixed a typo in UdfDriveBindingStop() ("This" -> "SimpleFs") which
>     broke retrieval of private fs data from SimpleFs protocol --
>     identified by 'reconnect -r' command in UEFI shell.

Follow the diff between v4 and v5 for Mde*Pkg changes (forgot to include 
it when resending):

diff --git a/MdeModulePkg/Universal/Disk/UdfDxe/File.c 
b/MdeModulePkg/Universal/Disk/UdfDxe/File.c
index 8ad14fe594..2dbcff0be4 100644
--- a/MdeModulePkg/Universal/Disk/UdfDxe/File.c
+++ b/MdeModulePkg/Universal/Disk/UdfDxe/File.c
@@ -372,7 +372,7 @@ UdfRead (
        PrivFileData->FileSize,
        &PrivFileData->FilePosition,
        Buffer,
-      BufferSize
+      (UINT64 *)(UINTN)BufferSize^M
        );
    } else if (IS_FID_DIRECTORY_FILE (Parent->FileIdentifierDesc)) {
      if (ReadDirInfo->FidOffset == 0 && PrivFileData->FilePosition > 0) {
diff --git a/MdeModulePkg/Universal/Disk/UdfDxe/Udf.c 
b/MdeModulePkg/Universal/Disk/UdfDxe/Udf.c
index 9f10c78ca9..49dc7077b7 100644
--- a/MdeModulePkg/Universal/Disk/UdfDxe/Udf.c
+++ b/MdeModulePkg/Universal/Disk/UdfDxe/Udf.c
@@ -264,7 +264,7 @@ UdfDriverBindingStop (
      EFI_OPEN_PROTOCOL_GET_PROTOCOL
      );
    if (!EFI_ERROR (Status)) {
-    PrivFsData = PRIVATE_UDF_SIMPLE_FS_DATA_FROM_THIS (This);
+    PrivFsData = PRIVATE_UDF_SIMPLE_FS_DATA_FROM_THIS (SimpleFs);^M

      //
      // Uninstall child handle


Thanks,
Paulo