X-Git-Url: https://git.proxmox.com/?p=mirror_edk2.git;a=blobdiff_plain;f=SecurityPkg%2FUserIdentification%2FUserProfileManagerDxe%2FUserProfileManager.c;h=4b1eabaaed0d50375c4fc89bdd098657604d9d13;hp=6e5bd06502140399f023c3c1c5f348c54e6b9978;hb=fab1046560e520486f255093cbb4fe36a347f93a;hpb=639a76d1912f8eb07e26b1a8f2393b682be65f2c

diff --git a/SecurityPkg/UserIdentification/UserProfileManagerDxe/UserProfileManager.c b/SecurityPkg/UserIdentification/UserProfileManagerDxe/UserProfileManager.c
index 6e5bd06502..4b1eabaaed 100644
--- a/SecurityPkg/UserIdentification/UserProfileManagerDxe/UserProfileManager.c
+++ b/SecurityPkg/UserIdentification/UserProfileManagerDxe/UserProfileManager.c
@@ -309,7 +309,7 @@ UserProfileManagerCallback (
     Status = EFI_SUCCESS;
     break;
 
-  case EFI_BROWSER_ACTION_CHANGING:
+  case EFI_BROWSER_ACTION_CHANGED:
   {  
     //
     // Handle the request from form.
@@ -337,13 +337,6 @@ UserProfileManagerCallback (
       // Judge next 2 bits.
       //
       switch (QuestionId & KEY_SECOND_FORM_MASK) {
-      //
-      // Enter delete user profile form.
-      //
-      case KEY_ENTER_NEXT_FORM:
-        SelectUserToDelete ();
-        break;
-
       //
       // Delete specified user profile.
       //
@@ -368,13 +361,6 @@ UserProfileManagerCallback (
       // Judge next 2 bits.
       //
       switch (QuestionId & KEY_SECOND_FORM_MASK) {
-      //
-      // Enter modify user profile form.
-      //
-      case KEY_ENTER_NEXT_FORM:
-        SelectUserToModify ();
-        break;
-
       //
       // Enter user profile information form.
       //
@@ -383,13 +369,6 @@ UserProfileManagerCallback (
         // Judge next 3 bits.
         //
         switch (QuestionId & KEY_MODIFY_INFO_MASK) {
-        //
-        // Display user information form.
-        //
-        case KEY_ENTER_NEXT_FORM:
-          ModifyUserInfo ((UINT8) QuestionId);
-          break;
-
         //
         // Modify user name.
         //
@@ -409,13 +388,6 @@ UserProfileManagerCallback (
           // Judge next 3 bits
           //
           switch (QuestionId & KEY_MODIFY_IP_MASK) {
-          //
-          // Display identity policy modify form.
-          //
-          case KEY_ENTER_NEXT_FORM:
-            ModifyIdentityPolicy ();
-            break;
-
           //
           // Change credential provider option.
           //
@@ -442,7 +414,7 @@ UserProfileManagerCallback (
           //
           case KEY_IP_RETURN_UIF:
             SaveIdentityPolicy ();
-            *ActionRequest = EFI_BROWSER_ACTION_REQUEST_FORM_APPLY;
+            *ActionRequest = EFI_BROWSER_ACTION_REQUEST_FORM_SUBMIT_EXIT;
             break;
 
           default:
@@ -458,13 +430,6 @@ UserProfileManagerCallback (
           // Judge next 3 bits.
           //
           switch (QuestionId & KEY_MODIFY_AP_MASK) {
-          //
-          // Display access policy modify form.
-          //
-          case KEY_ENTER_NEXT_FORM:
-            ModidyAccessPolicy ();
-            break;
-
           //
           // Change access right choice.
           //
@@ -545,7 +510,7 @@ UserProfileManagerCallback (
           //
           case KEY_AP_RETURN_UIF:
             SaveAccessPolicy ();
-            *ActionRequest = EFI_BROWSER_ACTION_REQUEST_FORM_APPLY;
+            *ActionRequest = EFI_BROWSER_ACTION_REQUEST_FORM_SUBMIT_EXIT;
             break;
 
           default:
@@ -642,6 +607,126 @@ UserProfileManagerCallback (
   }
   break;
 
+
+  case EFI_BROWSER_ACTION_CHANGING:
+  {  
+    //
+    // Handle the request from form.
+    //
+    if (Value == NULL) {
+      return EFI_INVALID_PARAMETER;
+    }
+    
+    //
+    // Judge first 2 bits.
+    //
+    switch (QuestionId & KEY_FIRST_FORM_MASK) {
+    //
+    // Delete user profile operation.
+    //
+    case KEY_DEL_USER:
+      //
+      // Judge next 2 bits.
+      //
+      switch (QuestionId & KEY_SECOND_FORM_MASK) {
+      //
+      // Enter delete user profile form.
+      //
+      case KEY_ENTER_NEXT_FORM:
+        SelectUserToDelete ();
+        break;
+
+      default:
+        break;
+      }
+      break;
+
+    //
+    // Modify user profile operation.
+    //
+    case KEY_MODIFY_USER:
+      //
+      // Judge next 2 bits.
+      //
+      switch (QuestionId & KEY_SECOND_FORM_MASK) {
+      //
+      // Enter modify user profile form.
+      //
+      case KEY_ENTER_NEXT_FORM:
+        SelectUserToModify ();
+        break;
+
+      //
+      // Enter user profile information form.
+      //
+      case KEY_SELECT_USER:
+        //
+        // Judge next 3 bits.
+        //
+        switch (QuestionId & KEY_MODIFY_INFO_MASK) {
+        //
+        // Display user information form.
+        //
+        case KEY_ENTER_NEXT_FORM:
+          ModifyUserInfo ((UINT8) QuestionId);
+          break;
+
+        //
+        // Modify identity policy.
+        //
+        case KEY_MODIFY_IP:
+          //
+          // Judge next 3 bits
+          //
+          switch (QuestionId & KEY_MODIFY_IP_MASK) {
+          //
+          // Display identity policy modify form.
+          //
+          case KEY_ENTER_NEXT_FORM:
+            ModifyIdentityPolicy ();
+            break;
+
+          default:
+            break;
+          }
+          break;
+
+        //
+        // Modify access policy.
+        //
+        case KEY_MODIFY_AP:
+          //
+          // Judge next 3 bits.
+          //
+          switch (QuestionId & KEY_MODIFY_AP_MASK) {
+          //
+          // Display access policy modify form.
+          //
+          case KEY_ENTER_NEXT_FORM:
+            ModidyAccessPolicy ();
+            break;
+
+          default:
+            break;
+          }
+          break;
+
+        default:
+          break;
+        }
+        break;
+
+      default:
+        break;
+      }
+      break;
+
+    default:
+      break;
+    }
+  }
+  break;
+
   default:
     //
     // All other action return unsupported.