Fix: eliminate buffer overflow in countFields() and add callers error handling

Author: ayushman1210

src/common/modelParams.c

@@ -45,7 +45,7 @@ void checkAllRead(ModelParams *ModelParams) {
     if (!param->isRead) {
       if (param->isRequired) {  // should have been read but wasn't!
         okay = 0;
-        logError("Did not find read required parameter %s\n", param->name);
+        logError("Did not find required parameter %s\n", param->name);
       } else {
         missingOptParam = 1;
       }
@@ -127,6 +127,10 @@ void initializeOneModelParam(ModelParams *modelParams, char *name,
 
 void checkParamFormat(char *line, const char *sep) {
   int numParams = countFields(line, sep);
+  if (numParams == -1) {
+    logError("memory allocation failure in parameter file processing\n");
+    exit(EXIT_CODE_INTERNAL_ERROR);
+  }
   if (numParams > 2) {
     logWarning("extra columns in .param file are being ignored (found %d "
                "columns)\n",

src/common/util.c

@@ -50,13 +50,19 @@ int stripComment(char *line, const char *commentChars) {
 // count number of fields in a string separated by delimiter 'sep'
 int countFields(const char *line, const char *sep) {
   // strtok modifies string, so we need a copy
-  char lineCopy[256];
+  size_t lineLen = strlen(line);
+  char *lineCopy = (char *)malloc(lineLen + 1);
+  if (lineCopy == NULL) {
+    return -1; // Handle allocation failure
+  }
   strcpy(lineCopy, line);
+  
   int numParams = 0;
   char *par = strtok(lineCopy, sep);
   while (par != NULL) {
     ++numParams;
     par = strtok(NULL, sep);
   }
+  free(lineCopy);
   return numParams;
 }