diff options
author | Zou Nan hai <nanhai.zou@intel.com> | 2006-02-06 22:25:55 -0500 |
---|---|---|
committer | Tony Luck <tony.luck@intel.com> | 2006-02-07 13:59:37 -0500 |
commit | ecdd5dabd33d67066d476467e447cdcadab90550 (patch) | |
tree | c2387dcd3e1926193cec6db89ceac75dd63e207c /arch/ia64 | |
parent | d43da75fd6be4197c280903e1553eafcc39218e2 (diff) |
[IA64] Fix a possible buffer overflow in efi.c
Make sure to save space for the trailing '\0'.
Signed-off-by: Zou Nan hai <nanhai.zou@intel.com>
Signed-off-by: Tony Luck <tony.luck@intel.com>
Diffstat (limited to 'arch/ia64')
-rw-r--r-- | arch/ia64/kernel/efi.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/ia64/kernel/efi.c b/arch/ia64/kernel/efi.c index c485a3b32ba8..d51c2f214875 100644 --- a/arch/ia64/kernel/efi.c +++ b/arch/ia64/kernel/efi.c | |||
@@ -458,7 +458,7 @@ efi_init (void) | |||
458 | /* Show what we know for posterity */ | 458 | /* Show what we know for posterity */ |
459 | c16 = __va(efi.systab->fw_vendor); | 459 | c16 = __va(efi.systab->fw_vendor); |
460 | if (c16) { | 460 | if (c16) { |
461 | for (i = 0;i < (int) sizeof(vendor) && *c16; ++i) | 461 | for (i = 0;i < (int) sizeof(vendor) - 1 && *c16; ++i) |
462 | vendor[i] = *c16++; | 462 | vendor[i] = *c16++; |
463 | vendor[i] = '\0'; | 463 | vendor[i] = '\0'; |
464 | } | 464 | } |