ShellPkg: DXE_ASSERT when issuing drvcfg command in UEFI Shell

[zhurui22]

Fixes #10626

Issue command drvcfg derectly, system halt

Cc: Liming Gao gaoliming@byosoft.com.cn
Reviewed-by: Liming Gao gaoliming@byosoft.com.cn

Description

<Include a description of the change and why this change was made.>

<For each item, place an "x" in between [ and ] if true. Example: [x] (you can also check items in GitHub UI)>

<Create the PR as a Draft PR if it is only created to run CI checks.>

<Delete lines in <> tags before creating the PR.>

• Breaking change?
  • Breaking change - Does this PR cause a break in build or boot behavior?
  • Examples: Does it add a new library class or move a module to a different repo.
• Impacts security?
  • Security - Does this PR have a direct security impact?
  • Examples: Crypto algorithm change or buffer overflow fix.
• Includes tests?
  • Tests - Does this PR include any explicit test code?
  • Examples: Unit tests or integration tests.

How This Was Tested

<Describe the test(s) that were run to verify the changes.>

Integration Instructions

<Describe how these changes should be integrated. Use N/A if nothing is required.>

[lgao4]

With this change, drvcfg will output help message when no input is specified?

[pierregondois]

With this change, drvcfg will output help message when no input is specified?

It seems the command drvcfg alone is valid, so it might not be necessary to output help message.
However issuing drvcfg -c doesn't output anything for me even though all devices and child devices should be printed out.

To display the list of devices that are available for configuration:
Shell> drvcfg
To display the list of devices and child devices that are available for configuration:
Shell> drvcfg –c

[lgao4]

drvcfg

According to UEFI Shell spec, please make sure this command works.

To display the list of devices that are available for configuration:
Shell> drvcfg
To display the list of devices and child devices that are available for configuration:
Shell> drvcfg -c

[pierregondois]

With this change, drvcfg will output help message when no input is specified?

It seems the command drvcfg alone is valid, so it might not be necessary to output help message. However issuing drvcfg -c doesn't output anything for me even though all devices and child devices should be printed out.

To display the list of devices that are available for configuration:
Shell> drvcfg
To display the list of devices and child devices that are available for configuration:
Shell> drvcfg –c

Just in case, this patch is an improvement compared to the current state where drvcfg -c crashes,
it's just that it would be even better to have a command doing something usefull.

[lgao4]

With this change, drvcfg will output help message when no input is specified?

It seems the command drvcfg alone is valid, so it might not be necessary to output help message. However issuing drvcfg -c doesn't output anything for me even though all devices and child devices should be printed out.

To display the list of devices that are available for configuration:
Shell> drvcfg
To display the list of devices and child devices that are available for configuration:
Shell> drvcfg –c

Just in case, this patch is an improvement compared to the current state where drvcfg -c crashes, it's just that it would be even better to have a command doing something usefull.

Agree this change is better than before.

[pierregondois]

The commit message format is not valid:
 * First line of commit message (subject line) is too long (92 >= 76).

Otherwise lgtm aswell