X-Git-Url: https://plomlompom.com/repos/%7B%7B%20web_path%20%7D%7D/decks/%7B%7Bdeck_id%7D%7D/cards/%7B%7Bcard_id%7D%7D/form?a=blobdiff_plain;f=src%2Fclient%2Fcommand_db.c;h=19cd4064e09c7878d49d841ed1e86d30536ffc58;hb=d61da76579b416799feb44bfb9a1e75ce1a58067;hp=a74a93b1984014f7132f5a345c048eab2bc37cc2;hpb=b6c140e3bd19ede26e3992c03c576515487f0a10;p=plomrogue

diff --git a/src/client/command_db.c b/src/client/command_db.c
index a74a93b..19cd406 100644
--- a/src/client/command_db.c
+++ b/src/client/command_db.c
@@ -3,7 +3,7 @@
 #include "command_db.h"
 #include <stddef.h> /* NULL */
 #include <stdint.h> /* uint8_t, uint32_t */
-#include <stdio.h> /* FILE */
+#include <stdio.h> /* FILE, sprintf() */
 #include <stdlib.h> /* free() */
 #include <string.h> /* memcpy(), strlen(), strtok(), strcmp() */
 #include "../common/readwrite.h" /* try_fopen(), try_fclose(), try_fgets()
@@ -12,21 +12,43 @@
 #include "../common/rexit.h" /* for exit_err() */
 #include "../common/try_malloc.h" /* try_malloc() */
 #include "cleanup.h" /* set_cleanup_flag() */
+#include "misc.h" /* array_append() */
 #include "world.h" /* global world */
 
 
 
-/* Point "ch_ptr" to next strtok() string in "line" delimited by "delim".*/
-static void copy_tokenized_string(char * line, char ** ch_ptr, char * delim);
+/* Helpers to init_command_db(). */
+static uint8_t copy_tokenized_string(char * line, char ** ch_ptr, char * delim);
+static char * init_command_db_err(char * line_copy, uint8_t line_number);
 
 
-
-static void copy_tokenized_string(char * line, char ** ch_ptr, char * delim)
+static uint8_t copy_tokenized_string(char * line, char ** ch_ptr, char * delim)
 {
     char * f_name = "copy_tokenized_string()";
     char * dsc_ptr = strtok(line, delim);
+    if (!dsc_ptr)
+    {
+        return 1;
+    }
     * ch_ptr = try_malloc(strlen(dsc_ptr) + 1, f_name);
     memcpy(* ch_ptr, dsc_ptr, strlen(dsc_ptr) + 1);
+    return 0;
+}
+
+
+
+static char * init_command_db_err(char * line_copy, uint8_t line_number)
+{
+    char * f_name = "init_command_db_err";
+    char * err_start = "Failed reading command config file at ";
+    char * err_middle = " due to malformed line ";
+    line_copy[strlen(line_copy) - 1] = '\0';
+    char * err = try_malloc(strlen(err_start) + strlen(world.path_commands) +
+                            strlen(err_middle) + 3 + 2 + strlen(line_copy) + 1,
+                            f_name);
+    sprintf(err, "%s%s%s%d: %s", err_start, world.path_commands, err_middle,
+            line_number, line_copy);
+    return err;
 }
 
 
@@ -60,7 +82,6 @@ extern void init_command_db()
     uint32_t lines;
     uint32_t linemax = textfile_sizes(file, &lines);
     char line[linemax + 1];
-    world.commandDB.cmds = try_malloc(lines * sizeof(struct Command), f_name);
     uint8_t i = 0;
     char * delim = " ";
     while (try_fgets(line, linemax + 1, file, f_name))
@@ -69,20 +90,28 @@ extern void init_command_db()
         {
             break;
         }
-        copy_tokenized_string(line, &world.commandDB.cmds[i].dsc_short, delim);
-        copy_tokenized_string(NULL, &world.commandDB.cmds[i].server_msg, delim);
-        if (!strcmp("0", world.commandDB.cmds[i].server_msg))
-        {                                             /*.server_msg==0 detects*/
-            free(world.commandDB.cmds[i].server_msg); /* non-server commands  */
-            world.commandDB.cmds[i].server_msg = NULL;/* in try_key() /       */
-        }                                             /* try_server_command().*/
-        char * arg_string = strtok(NULL, delim);
-        world.commandDB.cmds[i].arg = arg_string[0];
-        copy_tokenized_string(NULL, &world.commandDB.cmds[i].dsc_long, "\n");
+        char line_copy[strlen(line) + 1];
+        sprintf(line_copy, "%s", line);
+        struct Command cmd;
+        char * arg_string;
+        exit_err((  copy_tokenized_string(line, &cmd.dsc_short, delim)
+                 || copy_tokenized_string(NULL, &cmd.server_msg, delim)
+                 || NULL == (arg_string = strtok(NULL, delim))
+                 || strlen(arg_string) > 1
+                 || copy_tokenized_string(NULL, &cmd.dsc_long, "\n")),
+                 init_command_db_err(line_copy, i + 1));
+        cmd.arg = arg_string[0];
+        if (!strcmp("0", cmd.server_msg))
+        {                          /* A .server_msg == NULL helps control.c's */
+            free(cmd.server_msg);  /* try_key() and try_server_command() to   */
+            cmd.server_msg = NULL; /* differentiate server commands from      */
+        }                          /* non-server commands.                    */
+        array_append(i, sizeof(struct Command), (void *) &cmd,
+                     (void **) &world.commandDB.cmds);
         i++;
     }
     try_fclose(file, f_name);
-    world.commandDB.n = lines;
+    world.commandDB.n = i;
     set_cleanup_flag(CLEANUP_COMMANDS);
 }