OvmfPkg/VirtioFsDxe: manage path lifecycle in OpenVolume, Close, Delete

Add a canonical pathname field to VIRTIO_FS_FILE.

Initialize the new field in EFI_SIMPLE_FILE_SYSTEM_PROTOCOL.OpenVolume().

Release the new field in EFI_FILE_PROTOCOL.Close() and
EFI_FILE_PROTOCOL.Delete().

Cc: Ard Biesheuvel <ard.biesheuvel@arm.com>
Cc: Jordan Justen <jordan.l.justen@intel.com>
Cc: Philippe Mathieu-Daudé <philmd@redhat.com>
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=3097
Signed-off-by: Laszlo Ersek <lersek@redhat.com>
Message-Id: <20201216211125.19496-18-lersek@redhat.com>
Acked-by: Ard Biesheuvel <ard.biesheuvel@arm.com>

1 files changed, 12 insertions, 1 deletions

diff --git a/OvmfPkg/VirtioFsDxe/SimpleFsOpenVolume.c b/OvmfPkg/VirtioFsDxe/SimpleFsOpenVolume.c
index 67d2deb6bd..9c0ab434c1 100644
--- a/OvmfPkg/VirtioFsDxe/SimpleFsOpenVolume.c
+++ b/OvmfPkg/VirtioFsDxe/SimpleFsOpenVolume.c
@@ -28,6 +28,7 @@ VirtioFsOpenVolume (
   VIRTIO_FS      *VirtioFs;

   VIRTIO_FS_FILE *VirtioFsFile;

   EFI_STATUS     Status;

+  CHAR8          *CanonicalPathname;

   UINT64         RootDirHandle;

 

   VirtioFs = VIRTIO_FS_FROM_SIMPLE_FS (This);

@@ -37,13 +38,19 @@ VirtioFsOpenVolume (
     return EFI_OUT_OF_RESOURCES;

   }

 

+  CanonicalPathname = AllocateCopyPool (sizeof "/", "/");

+  if (CanonicalPathname == NULL) {

+    Status = EFI_OUT_OF_RESOURCES;

+    goto FreeVirtioFsFile;

+  }

+

   //

   // Open the root directory.

   //

   Status = VirtioFsFuseOpenDir (VirtioFs, VIRTIO_FS_FUSE_ROOT_DIR_NODE_ID,

              &RootDirHandle);

   if (EFI_ERROR (Status)) {

-    goto FreeVirtioFsFile;

+    goto FreeCanonicalPathname;

   }

 

   //

@@ -64,6 +71,7 @@ VirtioFsOpenVolume (
   VirtioFsFile->IsDirectory            = TRUE;

   VirtioFsFile->IsOpenForWriting       = FALSE;

   VirtioFsFile->OwnerFs                = VirtioFs;

+  VirtioFsFile->CanonicalPathname      = CanonicalPathname;

   VirtioFsFile->NodeId                 = VIRTIO_FS_FUSE_ROOT_DIR_NODE_ID;

   VirtioFsFile->FuseHandle             = RootDirHandle;

 

@@ -75,6 +83,9 @@ VirtioFsOpenVolume (
   *Root = &VirtioFsFile->SimpleFile;

   return EFI_SUCCESS;

 

+FreeCanonicalPathname:

+  FreePool (CanonicalPathname);

+

 FreeVirtioFsFile:

   FreePool (VirtioFsFile);