diff options
| author | Elvin Li <elvin.li@intel.com> | 2014-08-01 05:08:40 +0000 |
|---|---|---|
| committer | li-elvin <li-elvin@6f19259b-4bc3-4df7-8a09-765794883524> | 2014-08-01 05:08:40 +0000 |
| commit | ffbc792ce5edf8ec6250c9808dc4bcad8cfd08b8 (patch) | |
| tree | b16f4c4ffdda08de9d0b013743f8e0ead87e3de4 | |
| parent | cebd6ef9048067ab53ab03e344f9df2cb506080d (diff) | |
| download | edk2-platforms-ffbc792ce5edf8ec6250c9808dc4bcad8cfd08b8.tar.xz | |
MdeModulePkg: Fix wrong check for SetAttribute in Consplitter
Original code check if Attribute > 0x7FFFFFFF, this is wrong and fail to check valid case per UEFI spec.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Elvin Li <elvin.li@intel.com>
Reviewed-by: Jaben Carsey <Jaben.carsey@intel.com>
git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@15734 6f19259b-4bc3-4df7-8a09-765794883524
| -rw-r--r-- | MdeModulePkg/Universal/Console/ConSplitterDxe/ConSplitter.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/MdeModulePkg/Universal/Console/ConSplitterDxe/ConSplitter.c b/MdeModulePkg/Universal/Console/ConSplitterDxe/ConSplitter.c index a0a2dd3358..bea206d35b 100644 --- a/MdeModulePkg/Universal/Console/ConSplitterDxe/ConSplitter.c +++ b/MdeModulePkg/Universal/Console/ConSplitterDxe/ConSplitter.c @@ -16,7 +16,7 @@ never removed. Such design ensures sytem function well during none console
device situation.
-Copyright (c) 2006 - 2012, Intel Corporation. All rights reserved.<BR>
+Copyright (c) 2006 - 2014, Intel Corporation. All rights reserved.<BR>
This program and the accompanying materials
are licensed and made available under the terms and conditions of the BSD License
which accompanies this distribution. The full text of the license may be found at
@@ -4557,7 +4557,7 @@ ConSplitterTextOutSetAttribute ( //
// Check whether param Attribute is valid.
//
- if ( (Attribute > (UINTN)(((UINT32)-1)>>1)) ) {
+ if ((Attribute | 0x7F) != 0x7F) {
return EFI_UNSUPPORTED;
}
|