Problems writing to 2-Dimensional arrays, Embedded C - c

Question

I am having trouble understanding what is going on with my code, mainly regarding 2D arrays. I am trying to use a function in my communication file to decode a command string from a terminal. I want it to break the string down and output the answers by updating the global buffers. I am having trouble writing to these arrays and I do not understand why. In the code below I just assigned a value to the USART1_Command char array where in the actual program it comes from a terminal.
Main and declarations:
//Global declarations//
#define USART1_BUFFER_SIZE (100)
#define MAX_COMMAND_INT_VALUES (50)
#define MAX_COMMAND_STR_VALUES (50)
#define MAX_COMMAND_STR_LENGTH (50)
char USART1_Buffer[USART1_BUFFER_SIZE];
uint16_t USART1_Write_Index = 0;
char USART1_Command[MAX_COMMAND_STR_LENGTH] = {0x00};
uint16_t USART1_Command_Int_Values[MAX_COMMAND_INT_VALUES] = {0x00};
char **USART1_Command_Str_Values;
void main(void){
USART1_Command_Str_Values = (char **)calloc(MAX_COMMAND_STR_VALUES, sizeof(char *));
uint16_t i = 0;
for(i = 0; i < MAX_COMMAND_INT_VALUES; i++){
USART1_Command_Str_Values[i] = (char *)calloc(MAX_COMMAND_STR_LENGTH, sizeof(char));
}
strcpy(USART1_Command, "TESTCOMMAND,45,36,21-TEST1-TEST2");
USART1_Command_Decode(USART1_Command, USART1_Command_Int_Values, USART1_Command_Str_Values, MAX_COMMAND_INT_VALUES, MAX_COMMAND_STR_VALUES, MAX_COMMAND_STR_LENGTH);
while(1){
}
}
and the function:
void USART1_Command_Decode(char *command, uint16_t *int_values_buffer, char **str_values_buffer, uint16_t int_values_buffer_size, uint16_t str_values_buffer_size, uint16_t str_values_size){
uint16_t i = 0;
uint16_t j = 0;
uint16_t length = strlen(command);
int16_t Int_Values_Index = -1;
bool Int_Value_Flag = false;
char **Int_Values;
Int_Values = (char**)calloc(int_values_buffer_size, sizeof(char*));
for(i = 0; i < int_values_buffer_size; i++){
Int_Values[i] = (char*)calloc(str_values_size, sizeof(char));
}
const uint16_t Max_Str_Values = 50;
int16_t Str_Values_Index = -1;
bool Str_Value_Flag = false;
for(i = 0; i < length; i++){
switch(command[i]){
case ',': //Signifies integer value//
if(Int_Values_Index < int_values_buffer_size){
Int_Values_Index++;
Int_Value_Flag = true;
Str_Value_Flag = false;
j = 0;
}
command[i] = 0x00;
break;
case '-': //Signifies string value//
if(Str_Values_Index < str_values_buffer_size){
Str_Values_Index++;
Str_Value_Flag = true;
Int_Value_Flag = false;
j = 0;
}
command[i] = 0x00;
break;
default:
if(Int_Value_Flag){
Int_Values[Int_Values_Index][j] = command[i];
command[i] = 0x00;
j++;
}
if(Str_Value_Flag){
str_values_buffer[Str_Values_Index][j] = command[i];
command[i] = 0x00;
j++;
}
break;
}
}
//Convert integer strings to integers//
for(i = 1; i <= Int_Values_Index; i++){
int_values_buffer[i] = atoi(Int_Values[i]);
}
for(i = 0; i < int_values_buffer_size; i++){
free(Int_Values[i]);
}
free(Int_Values);
}

Answer 1

I figured this out, the way that I was able to make it work was as follows:
Declaration of Global variables:
#define MAX_COMMAND_INT_VALUES (50)
#define MAX_COMMAND_STR_VALUES (50)
#define MAX_COMMAND_STR_LENGTH (50)
char USART1_Command[MAX_COMMAND_STR_LENGTH] = {0x00};
uint16_t USART1_Command_Int_Values[MAX_COMMAND_INT_VALUES] = {0x00};
char USART1_Command_Str_Values[MAX_COMMAND_STR_VALUES][MAX_COMMAND_STR_LENGTH];
Function:
void USART1_Command_Decode(char *command, uint16_t *int_values_buffer, uint16_t int_values_buffer_size, uint16_t str_values_buffer_size, uint16_t str_values_size, char str_values_buffer[str_values_buffer_size][str_values_size]){
uint16_t i = 0;
uint16_t j = 0;
uint16_t length = strlen(command);
int16_t Int_Values_Index = -1;
bool Int_Value_Flag = false;
char Int_Values[int_values_buffer_size][str_values_size];
for(i = 0; i < int_values_buffer_size; i++){
for(j = 0; j < str_values_size; j++){
Int_Values[i][j] = 0x00;
}
}
int16_t Str_Values_Index = -1;
bool Str_Value_Flag = false;
for(i = 0; i < length; i++){
switch(command[i]){
case ',': //Signifies integer value//
if(Int_Values_Index < int_values_buffer_size){
Int_Values_Index++;
Int_Value_Flag = true;
Str_Value_Flag = false;
j = 0;
}
command[i] = 0x00;
break;
case '-': //Signifies string value//
if(Str_Values_Index < str_values_buffer_size){
Str_Values_Index++;
Str_Value_Flag = true;
Int_Value_Flag = false;
j = 0;
}
command[i] = 0x00;
break;
default:
if(Int_Value_Flag){
Int_Values[Int_Values_Index][j] = command[i];
command[i] = 0x00;
j++;
}
if(Str_Value_Flag){
str_values_buffer[Str_Values_Index][j] = command[i];
command[i] = 0x00;
j++;
}
break;
}
}
//Convert integer strings to integers//
for(i = 0; i <= Int_Values_Index; i++){
int_values_buffer[i] = atoi(Int_Values[i]);
}
}
Function Call:
USART1_Command_Decode(USART1_Command, USART1_Command_Int_Values, MAX_COMMAND_INT_VALUES, MAX_COMMAND_STR_VALUES, MAX_COMMAND_STR_LENGTH, USART1_Command_Str_Values);
Thank you #P__J__ for the help!
EDIT:
I found the main problem with my original code, I was having problems with stack overflow because I was misunderstanding the allocation for stack and heap in Cross Studio. I had 256bytes of stack when I thought that I had more like 20KB. I allocated 16KB to stack and 8KB to heap, all the problems with malloc() and calloc() have disappeared along with the array writing issues. I apologize to everyone for the unnecessary question.