diff options
| author | Qiu Shumin <shumin.qiu@intel.com> | 2015-07-26 08:06:01 +0000 |
|---|---|---|
| committer | jljusten <jljusten@Edk2> | 2015-07-26 08:06:01 +0000 |
| commit | b0f81b62def9fa06b0eae9d7a6e9659e20c4afe7 (patch) | |
| tree | d4184a415a1edf4e0d2173db091f51c4a9405af6 /ShellPkg/Application | |
| parent | 41ab153f854986e401f6991ec1e507523a3b730e (diff) | |
| download | edk2-b0f81b62def9fa06b0eae9d7a6e9659e20c4afe7.zip edk2-b0f81b62def9fa06b0eae9d7a6e9659e20c4afe7.tar.gz edk2-b0f81b62def9fa06b0eae9d7a6e9659e20c4afe7.tar.bz2 | |
ShellPkg: Fix the ASSERT issue in Shell 'for' loop
The Length parameter of 'GetNextParameter' is the buffer size in bytes.
While StrnCpys requires user to pass the max number of dest unicode char,
we should convert size in bytes to the number of char.
Cc: Jaben Carsey <jaben.carsey@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Qiu Shumin <shumin.qiu@intel.com>
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com>
[lersek@redhat.com: updated commit message as requested by Jaben]
Signed-off-by: Laszlo Ersek <lersek@redhat.com>
git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@18059 6f19259b-4bc3-4df7-8a09-765794883524
Diffstat (limited to 'ShellPkg/Application')
| -rw-r--r-- | ShellPkg/Application/Shell/ShellParametersProtocol.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ShellPkg/Application/Shell/ShellParametersProtocol.c b/ShellPkg/Application/Shell/ShellParametersProtocol.c index de29c25..1c1367b 100644 --- a/ShellPkg/Application/Shell/ShellParametersProtocol.c +++ b/ShellPkg/Application/Shell/ShellParametersProtocol.c @@ -125,7 +125,7 @@ DEBUG_CODE_END(); return (EFI_NOT_FOUND);
}
- StrnCpyS(*TempParameter, Length, (*Walker), NextDelim - *Walker);
+ StrnCpyS(*TempParameter, Length / sizeof(CHAR16), (*Walker), NextDelim - *Walker);
//
// Add a CHAR_NULL if we didnt get one via the copy
|