183b83e208
Backport some lcdproc patches from upstream, to fix the following issues. * Incorrect parsing of minus key in handler for client_add_key command * Segfault in handler for menu_add_item command Signed-off-by: Simon Dawson <spdawson@gmail.com> Signed-off-by: Peter Korsgaard <jacmet@sunsite.dk>
20 lines
876 B
Diff
20 lines
876 B
Diff
Trying to add a menu item using menu_add_item causes a segmentation fault
|
|
in the LCDd server. This is caused by a read beyond the end of an array.
|
|
|
|
This patch adds a safety check on the size of the argument array.
|
|
|
|
Signed-off-by: Simon Dawson <spdawson@gmail.com>
|
|
|
|
diff -Nurp a/server/commands/menu_commands.c b/server/commands/menu_commands.c
|
|
--- a/server/commands/menu_commands.c 2011-08-14 13:29:16.000000000 +0100
|
|
+++ b/server/commands/menu_commands.c 2012-10-04 22:19:07.997409193 +0100
|
|
@@ -198,7 +198,7 @@ menu_add_item_func(Client *c, int argc,
|
|
|
|
/* call menu_set_item() with a temporarily allocated argv
|
|
* to process the remaining options */
|
|
- if ((argc > 5) || (argv[4][0] == '-')) {
|
|
+ if ((argc > 5) || ((argc == 5) && argv[4][0] == '-')) {
|
|
// menu_add_item <menuid> <newitemid> <type> [<text>]
|
|
// menu_set_item <menuid> <itemid> {<option>}+
|
|
int i, j;
|