Changeset 44605 in vbox for trunk/src/VBox/Devices/EFI/Firmware2

Timestamp:

Feb 8, 2013 3:20:52 PM (12 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

83690

Message:

Some fixes and instrumentation

File:

• trunk/src/VBox/Devices/EFI/Firmware2/VBoxPkg/VBoxVariable/InitVariable.c (modified) (6 diffs)

trunk/src/VBox/Devices/EFI/Firmware2/VBoxPkg/VBoxVariable/InitVariable.c

@@ -148 +148 @@
      * Tell DevEFI to look for the specified variable.
      */
-    ASMOutU32(EFI_VARIABLE_OP, EFI_VM_VARIABLE_OP_GUID);
     VBoxWriteNVRAMGuidParam(VendorGuid);
     VBoxWriteNVRAMNameParam(VariableName);
-
     u32Rc = VBoxWriteNVRAMDoOp(EFI_VARIABLE_OP_QUERY);
     if (u32Rc == EFI_VARIABLE_OP_STATUS_OK)
@@ -232 +230 @@
     LogFlowFuncEnter();
 
+    if (!VariableNameSize || !VariableName || !VendorGuid)
+    {
+        LogFlowFuncLeaveRC(EFI_INVALID_PARAMETER);
+        return EFI_INVALID_PARAMETER;
+    }
+
     /* 
      * Tell DevEFI which the current variable is, then ask for the next one.
@@ -245 +249 @@
     if (u32Rc == EFI_VARIABLE_OP_STATUS_OK)
         u32Rc = VBoxWriteNVRAMDoOp(EFI_VARIABLE_OP_QUERY_NEXT);
+    /** @todo We're supposed to skip stuff depending on attributes and
+     *        runtime/boottime, at least if EmuGetNextVariableName is something
+     *        to go by... */
 
     if (u32Rc == EFI_VARIABLE_OP_STATUS_OK)
@@ -254 +261 @@
         ASMOutU32(EFI_VARIABLE_OP, EFI_VM_VARIABLE_OP_NAME_LENGTH_UTF16);
         cwcName = ASMInU32(EFI_VARIABLE_OP);
-        if (cwcName * 2 < *VariableNameSize) /* ASSUMES byte size is specified */
+        if ((cwcName + 1) * 2 <= *VariableNameSize) /* ASSUMES byte size is specified */
         {
             UINT32 i;
@@ -269 +276 @@
             VariableName[i] = '\0';
 
-            *VariableNameSize = cwcName * 2;
             rc = EFI_SUCCESS;
         }
         else
-        {
             rc = EFI_BUFFER_TOO_SMALL;
-            *VariableNameSize = (cwcName + 1) * 2;
-        }
+        *VariableNameSize = (cwcName + 1) * 2;
     }
     else
@@ -282 +286 @@
 
     LogFlowFuncLeaveRC(rc);
+    // Temporary - start
+    DebugPrint("GetNextVariableName: rc=%u cbName=%u Name=%s VendorGuid=%g\n", rc, *VariableNameSize, VariableName, VendorGuid);
+    // Temporary - end
     return rc;
 #endif