Ruiyu Ni
cb9f3a59c3
ShellPkg/DrvDiag: Fix a critical typo bug
...
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com >
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com >
Reviewed-by: Hao A Wu <hao.a.wu@intel.com >
2016-07-25 10:20:12 +08:00
Ruiyu Ni
95fc5a8775
ShellPkg: Fix echo to support displaying special characters
...
Run 'echo -t' without the patch will get the result:
echo: Unknown flag - '-t'
The expected result is to display '-t' literally.
This patch adds special handle for 'echo'. 'echo' will not use the
general parameter parsing library.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com >
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com >
2016-07-20 10:28:00 +08:00
Ruiyu Ni
0fcf8d4df8
ShellPkg: Add Shell[Get|Set]RawCmdLine to ShellCommandLib
...
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com >
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com >
2016-07-20 10:27:56 +08:00
Ruiyu Ni
c0bcd3433f
ShellPkg: Move FindFirstCharacter/GetNextParameter to ShellCommandLib
...
And add Shell prefix to the two library APIs.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com >
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com >
2016-07-20 10:27:51 +08:00
Ruiyu Ni
b464d7ed0f
ShellPkg/ShellCommandLib.h: Formalize EOL
...
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com >
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com >
2016-07-20 10:27:47 +08:00
Ruiyu Ni
cf041fd799
ShellPkg/UefiShellCommandLib.c: Formalize EOL
...
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com >
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com >
2016-07-20 10:26:58 +08:00
Ruiyu Ni
9ba25c7db7
ShellPkg: Fix VS2010/2012 build failure
...
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com >
Reviewed-by: Dandan Bi <dandan.bi@intel.com >
2016-07-19 11:12:22 +08:00
Ruiyu Ni
e7a5a238c3
ShellPkg/Ping6: Handle memory allocation failure
...
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com >
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com >
2016-07-18 11:04:22 +08:00
Ruiyu Ni
a915fea68e
ShellPkg/Ifconfig6: Handle memory allocation failure
...
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com >
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com >
2016-07-18 11:04:19 +08:00
Ruiyu Ni
eeb9744e56
ShellPkg/UefiShellLib.c: Handle memory allocation failure
...
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com >
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com >
2016-07-18 11:04:16 +08:00
Ruiyu Ni
18bff310aa
ShellPkg/IfConfig: Handle memory allocation failure
...
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com >
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com >
2016-07-18 11:04:13 +08:00
Ruiyu Ni
2efafabf41
ShellPkg/UefiShellLib: Handle memory allocation failure
...
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com >
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com >
2016-07-18 11:04:10 +08:00
Ruiyu Ni
28d447f9bd
ShellPkg/Mv: Handle memory allocation failure
...
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com >
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com >
2016-07-18 11:04:07 +08:00
Ruiyu Ni
0b34dc1324
ShellPkg/Cp: Handle memory allocation failure
...
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com >
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com >
2016-07-18 11:04:04 +08:00
Ruiyu Ni
a34b7c3662
ShellPkg/Cd: Handle memory allocation failure
...
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com >
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com >
2016-07-18 11:04:00 +08:00
Ruiyu Ni
581c03896c
ShellPkg/If: Handle memory allocation failure
...
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com >
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com >
2016-07-18 10:55:53 +08:00
Ruiyu Ni
59b7dbac4e
ShellPkg/For: Handle memory allocation failure
...
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com >
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com >
2016-07-18 10:55:48 +08:00
Ruiyu Ni
26ca45a86f
ShellPkg/DrvDiag: Handle memory allocation failure
...
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com >
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com >
2016-07-18 10:55:45 +08:00
Ruiyu Ni
58972f5cf7
ShellPkg/DrvCfg: Handle memory allocation failure
...
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com >
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com >
2016-07-18 10:55:42 +08:00
Ruiyu Ni
55b1d63de7
ShellPkg/DevTree: Handle memory allocation failure
...
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com >
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com >
2016-07-18 10:55:38 +08:00
Ruiyu Ni
3e9442a564
ShellPkg/Edit: Handle memory allocation failure
...
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com >
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com >
2016-07-18 10:55:35 +08:00
Ruiyu Ni
20cfed165c
ShellPkg/LoadPciRom: Handle memory allocation failure
...
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com >
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com >
2016-07-18 10:55:32 +08:00
Ruiyu Ni
aba0ca78c6
ShellPkg/EfiDecompress: Handle memory allocation failure
...
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com >
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com >
2016-07-18 10:55:29 +08:00
Ruiyu Ni
ca1b2411f5
ShellPkg/EfiCompress: Handle memory allocation failure
...
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com >
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com >
2016-07-18 10:55:26 +08:00
Ruiyu Ni
c87bb070a6
ShellPkg/DMem: Handle memory allocation failure
...
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com >
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com >
2016-07-18 10:55:23 +08:00
Ruiyu Ni
107d05a433
ShellPkg/UefiShellCommandLib.c: Handle memory allocation failure
...
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com >
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com >
2016-07-18 10:55:20 +08:00
Ruiyu Ni
2c7c3b87bf
ShellPkg/ConsistMapping.c: Handle memory allocation failure
...
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com >
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com >
2016-07-18 10:55:17 +08:00
Ruiyu Ni
5945813f69
ShellPkg/UefiShellBcfgCommandLib: Handle memory allocation failure
...
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com >
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com >
2016-07-18 10:55:14 +08:00
Ruiyu Ni
aa3276c171
ShellPkg/UefiHandleParsingLib.c: Handle memory allocation failure
...
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com >
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com >
2016-07-18 10:55:11 +08:00
Ruiyu Ni
d758f80971
ShellPkg/Dp: Handle memory allocation failure
...
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com >
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com >
2016-07-18 10:55:08 +08:00
Ruiyu Ni
9168df3dea
ShellPkg/ShellProtocol.c: Handle memory allocation failure
...
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com >
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com >
2016-07-18 10:55:05 +08:00
Ruiyu Ni
ffbc60a027
ShellPkg/ShellAddEnvVarToList: Handle memory allocation failure
...
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com >
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com >
2016-07-18 10:55:02 +08:00
Ruiyu Ni
31e5b912b9
ShellPkg/IsVolatileEnv: Handle memory allocation failure
...
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com >
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com >
2016-07-18 10:54:59 +08:00
Ruiyu Ni
b2c036a7f0
ShellPkg/Shell.c: Handle memory allocation failure
...
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com >
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com >
2016-07-18 10:54:56 +08:00
Zhang Lubo
e2f5c491d8
ShellPkg: Fix issue about Ifconfig6 -r command.
...
Follow the Shell Spec, when the interface name is Specified,
we need to refresh the Ipv6 configuration.
Cc: Hegde Nagaraj P <nagaraj-p.hegde@hpe.com >
Cc: Ye Ting <ting.ye@intel.com >
Cc: Fu Siyuan <siyuan.fu@intel.com >
Cc: Wu Jiaxin <jiaxin.wu@intel.com >
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Zhang Lubo <lubo.zhang@intel.com >
Reviewed-by: Hegde Nagaraj P <nagaraj-p.hegde@hpe.com >
Reviewed-by: Sriram Subramanian <sriram-s@hpe.com >
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com >
Reviewed-By: Wu Jiaxin <jiaxin.wu@intel.com >
2016-07-15 14:45:55 +08:00
Ruiyu Ni
664fd5877d
Revert "ShellPkg: Make the USB mouse behavior in 'edit' consistent with 'hexedit'."
...
This reverts commit ee60bd2b6a
and
e36ed7a84b
.
The two commits enhanced 'edit' to support text selection through
mouse. But the code introduced some bugs resulting the text
selection behavior is not very usable and potentially hang in
certain platforms.
So I'd like to revert them firstly. If there is strong
requirement for the text selection through mouse in 'edit'
I will refine and re-add the code.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com >
Reviewed-by: Jaben Carsey <jarben.carsey@intel.com >
2016-07-08 14:54:30 +08:00
Mudusuru, Giri P
1d32246161
ShellPkg: Series of patches to fix typos - apropriate to appropriate
...
Cc: Jaben Carsey <jaben.carsey@intel.com >
Cc: Shumin Qiu <shumin.qiu@intel.com >
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Giri P Mudusuru <giri.p.mudusuru@intel.com >
Reviewed-By: Jaben Carsey <jaben.carsey@intel.com >
2016-07-07 15:24:24 -07:00
Subramanian, Sriram (EG Servers Platform SW)
80f7a8f5f9
ShellPkg:Ping: Ping command hangs with intermittent packet losses
...
When we have intermittent loss of packets, ping command doesnt
return to Shell prompt. It keeps looping in the while (Status
==EFI_NOT_READY) since Private->RxCount will never reach
Private->SendNum. Addresses the issue with the use of a new varibale
in the PRIVATE structure.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Hegde, Nagaraj P <nagaraj-p.hegde@hpe.com >
Reviewed-by: Sriram Subramanian <sriram-s@hpe.com >
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com >
2016-07-07 08:00:50 -07:00
Thomas Palmer
e06a4c0812
Fix GUID dereference
...
Print's "%g" argument requires a EFI_GUID pointer, not the EFI_GUID
value.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Thomas Palmer <thomas.palmer@hpe.com >
Reviewed-By: Jaben Carsey <jaben.carsey@intel.com >
2016-07-06 08:19:54 -07:00
Zhang Lubo
dded3ae880
ShellPkg: Update the error prompt for ping6 command.
...
when we use the ping6 command without configuring the network
interface, it should give correct prompt to users.
Cc: Hegde Nagaraj P <nagaraj-p.hegde@hpe.com >
Cc: Fu Siyuan <siyuan.fu@intel.com >
Cc: Wu Jiaxin <jiaxin.wu@intel.com >
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Zhang Lubo <lubo.zhang@intel.com >
Reviewed-by: Sriram Subramanian <sriram-s@hpe.com >
Reviewed-by: Hegde, Nagaraj P <nagaraj-p.hegde@hpe.com >
Reviewed-By: Wu Jiaxin <jiaxin.wu@intel.com >
2016-07-05 16:48:31 +08:00
Laszlo Ersek
42cb906852
ShellPkg: UefiHandleParsingLib: remove tautological comparison
...
The code being removed in this patch dates back to git commit a405b86d27
(Sep 14, 2010; "udk2010.up2.shell initial release."). The condition always
evaluates to true, and it breaks DEBUG builds of ArmVirtPkg with gcc-6.1:
ShellPkg/Library/UefiHandleParsingLib/UefiHandleParsingLib.c:
In function 'ParseHandleDatabaseByRelationshipWithType':
ShellPkg/Library/UefiHandleParsingLib/UefiHandleParsingLib.c:2465:76:
error: self-comparison always evaluates to true
[-Werror=tautological-compare]
ASSERT((*HandleType)[HandleIndex] == (*HandleType)[HandleIndex]);
Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org >
Cc: Gerd Hoffmann <kraxel@redhat.com >
Cc: Jaben Carsey <jaben.carsey@intel.com >
Cc: Michael Zimmermann <sigmaepsilon92@gmail.com >
Reported-by: Gerd Hoffmann <kraxel@redhat.com >
Reported-by: Michael Zimmermann <sigmaepsilon92@gmail.com >
Suggested-by: Jaben Carsey <jaben.carsey@intel.com >
Ref: http://thread.gmane.org/gmane.comp.bios.edk2.devel/13794/focus=13939
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Laszlo Ersek <lersek@redhat.com >
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com >
2016-06-30 23:14:29 +02:00
Laszlo Ersek
4a7518d31a
ShellPkg: don't call functions with side effects in ASSERT_EFI_ERROR()
...
When ASSERT_EFI_ERROR() is compiled out, dependent on build flags, only
the status checking should be removed; the function calls should stay.
Cc: Jaben Carsey <jaben.carsey@intel.com >
Cc: Shumin Qiu <shumin.qiu@intel.com >
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Laszlo Ersek <lersek@redhat.com >
Reviewed-by: Giri P Mudusuru <giri.p.mudusuru@intel.com >
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com >
2016-06-30 17:27:15 +02:00
Ruiyu Ni
5e32460d80
ShellPkg/UefiHandleParsing: Show handle's loaded image device path
...
The patch fixed a bug in UefiHandleParsingLib to show handle's loaded
image device path.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com >
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com >
2016-06-21 15:59:26 +08:00
Star Zeng
46213c8eb4
ShellPkg: Replace UnicodeStrToAsciiStr/AsciiStrToUnicodeStr
...
It is the follow up of 3ab41b7a32
to replace UnicodeStrToAsciiStr/AsciiStrToUnicodeStr with
UnicodeStrToAsciiStrS/AsciiStrToUnicodeStrS.
Cc: Jiewen Yao <jiewen.yao@intel.com >
Cc: Jaben Carsey <jaben.carsey@intel.com >
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Star Zeng <star.zeng@intel.com >
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com >
2016-06-21 12:46:26 +08:00
Ruiyu Ni
e918029fbc
ShellPkg/UefiHandleParsingLib.c: Unify EOL and remove trailing space
...
The patch doesn't change any code logic.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com >
Cc: Jiewen Yao <jiewen.yao@intel.com >
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com >
2016-06-17 15:43:11 +08:00
Ruiyu Ni
2528e47cb2
ShellPkg: Use ".." to indicate device path is partially displayed
...
The output of "dh <handle>" originally is like below:
12C: USBIO DevicePath(x0)/Pci(0x14,0x0)/USB(0x6,0x0))
The device path part is very confusing. Use ".." in front of the
partially displayed device path is better and it also aligns to
the example output in Shell spec.
12C: USBIO DevicePath(..)/Pci(0x14,0x0)/USB(0x6,0x0))
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com >
Reviewed-by: Jiewen Yao <jiewen.yao@intel.com >
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com >
2016-06-17 15:43:11 +08:00
Marvin H?user
bd3fc8133b
ShellPkg/App: Fix memory leak and save resources.
...
1) RunSplitCommand() allocates the initial SplitStdOut via
CreateFileInterfaceMem(). Free SplitStdIn after the swap to fix
the memory leak.
2) In RunSplitCommand(), SplitStdOut is checked for equality with
StdIn. This cannot happen due to the if-check within the swap.
Hence remove it.
3) UefiMain() doesn't free SplitList. Delete all list entries and
reinitialize the list when in DEBUG. This does not include the
CreateFileInterfaceMem()-allocated SplitStd mentioned in 1), so
keep the ASSERT() until resolved.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Marvin Haeuser <Marvin.Haeuser@outlook.com >
Reviewed-by: Qiu Shumin <shumin.qiu@intel.com >
2016-05-25 10:16:46 +08:00
Fu Siyuan
6987d7aa83
ShellPkg: Fix ARM build failure.
...
This patch fixes the ARM build failure introduced by
2be45bfe27
.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Fu Siyuan <siyuan.fu@intel.com >
Reviewed-by: Ard Biesheuvel <ard.biesheuvel@linaro.org >
2016-05-20 14:43:09 +08:00
Qiu Shumin
f86a5c3465
ShellPkg/Bcfg: Add support for 'addp' command.
...
Until now the 'addp' command has been handled in the same way as 'add'.
Just copy the DevicePath starting from the Hard Drive node when 'addp'
is used.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Marvin Haeuser <Marvin.Haeuser@outlook.com >
Reviewed-by: Qiu Shumin <shumin.qiu@intel.com >
2016-05-20 14:16:49 +08:00
Marvin H?user
f3a51e989a
ShellPkg: Also accept gEfiUnicodeCollation2ProtocolGuid for parsing.
...
The EFI_UNICODE_COLLATION_PROTOCOL can have two different GUIDs.
Look for both to support more UEFI implementations.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Marvin Haeuser <Marvin.Haeuser@outlook.com >
Reviewed-by: Qiu Shumin <shumin.qiu@intel.com >
2016-05-20 14:16:47 +08:00