[riot-notifications] [RIOT-OS/RIOT] sys/shell: commands: Check argc in _pm_handler (#14005)
notifications at github.com
Sat May 2 22:11:20 CEST 2020
### Contribution description
I noticed that the command `pm` is now available in the shell when a board is flashed with `examples/default`. However, when I only entered `pm` and not `pm off`, I got a data bus error on a 6lowpan-clicker. On a pic32-wifire, the board reboots.
This is due to accessing args out-of-bound in function `_pm_handler` at line 153 and line 189.
### Testing procedure
Flash `examples/default` on a board. If you only type `pm`, the board should not crash/reboot. It should simply print how to use this command.
### Issues/PRs references
None. Found while working on #13820 .
You can view, comment on, or merge this pull request online at:
-- Commit Summary --
* sys/shell: commands: Check argc in _pm_handler
-- File Changes --
M sys/shell/commands/sc_pm.c (7)
-- Patch Links --
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the notifications