From: Pasi Kallinen <paxed@alt.org>
Date: Fri, 5 Feb 2021 09:14:11 +0000 (+0200)
Subject: Fix BIND segfault
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=9e550ccb078c112a4e12a57e3b5a719df122c0e1;p=nethack

Fix BIND segfault

Recent commit that added terminator entry to default_menu_cmd_info
caused a segfault when trying to BIND keys.
---

diff --git a/src/options.c b/src/options.c
index 0f28c5829..bd27dd166 100644
--- a/src/options.c
+++ b/src/options.c
@@ -487,7 +487,7 @@ check_misc_menu_command(char *opts, char *op UNUSED)
     const char *name_to_check;
 
     /* check for menu command mapping */
-    for (i = 0; i < SIZE(default_menu_cmd_info); i++) {
+    for (i = 0; default_menu_cmd_info[i].name; i++) {
         name_to_check = default_menu_cmd_info[i].name;
         if (match_optname(opts, name_to_check,
                           (int) strlen(name_to_check), TRUE))
@@ -6021,7 +6021,7 @@ parsebindings(char *bindings)
         return TRUE;
 
     /* is it a menu command? */
-    for (i = 0; i < SIZE(default_menu_cmd_info); i++) {
+    for (i = 0; default_menu_cmd_info[i].name; i++) {
         if (!strcmp(default_menu_cmd_info[i].name, bind)) {
             if (illegal_menu_cmd_key(key)) {
                 config_error_add("Bad menu key %s:%s", visctrl(key), bind);