C присвоение строки другой строке приводит к reallo c (): следующий размер неверен - PullRequest
0 голосов
/ 07 апреля 2020

Я пытаюсь присвоить строку в структуре строке другой структуры:

func->parameters[func->parameter_amount].name = tokens[i+1].value;

это происходит через некоторое время l oop:

while (true) {
    func->parameter_amount++;
    func->parameters = realloc(func->parameters, func-> parameter_amount*sizeof(struct parameter));
    if (func->parameters == NULL) {
        mem_error();
    }
    if ((tokens[i].token != symbol) && (tokens[i].token != comma)) {
        break;
    } else if ((tokens[i].token == symbol) && tokens[i+1].token == symbol) {
        func->parameters[func->parameter_amount].type = string_to_type(tokens[i].value);
        if (func->parameters[func->parameter_amount].type == -1) {
            printf("Error: Invalid type '%s' for parameter declaration in function '%s' on line %i\n", tokens[i].value, func->name, func->line);
            exit(1);
        }
        func->parameters[func->parameter_amount].name = tokens[i+1].value;
        i += 2;
    } else if (tokens[i].token == comma) {
        func->parameter_amount--;
        i++;
    }
}

после происходит присвоение, программа говорит: realloc(): invalid next size и прерывает

структуры определены как:

struct parameter {
    int type;
    char* name;
};

struct function {
    int line;
    int type;
    char* name;
    struct parameter* parameters;
    int parameter_amount;
};

typedef struct {
    int token;
    char* value;
    int line;
} token;

Я не могу понять, что происходит не так

1 Ответ

1 голос
/ 07 апреля 2020

В вашем коде несколько ошибок:

1) После realloc размер массива func->parameters равен func->parameterAmount. Таким образом, это означает, что последний индекс, который вы можете использовать func->parameterAmount-1:

func->parameters[func->parameterAmount - 1]

2) Для каждого элемента в массиве func->parameters, вы должны выделить строку value (потому что в этот момент value это просто указатель на символ):

int i = 0;
int n = 129; // n will be the max length (minus 1) of newly allocated string
for (i = 0; i < func->parameterAmount; ++i) {
    func->parameters[i].name = (char *) malloc(sizeof(char) * n);
    if (func->parameters[i].name == NULL) {
      // Handle alloc error
    }
} 

Кроме того, не забудьте выделить все строковые переменные value внутри токена array.

3) В C вы не может присвоить значение строке таким образом. Вы должны использовать strcpy() из string.h header:

strcpy(func->parameters[func->parameter_amount].name, tokens[i+1].value);
...