0

На строке: "line = (char*)realloc(line, (i+1)*sizeof(char));" вылетает, если ввести больше чем 80 символов.

gdb вытаёт это: Что выдаёт gdb Я ввёл 127 символов и произошла ошибка, причём line выводится корректно.

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#define N 80
#define ERRORallocateLine "Failed allocate line successfully...\n"
#define ERRORallocateFlights "Failed allocate flights successfully...\n"

typedef struct Flight
{
    int id; // ID рейса
    int quantityPassengers; // Кол-во пассажиров
    char *from; // Адрес начала строки, содержащую информацию откуда будет выполняться рейс
    char *target; // Адрес начала строки, содержащую информацию куда будет направляться рейс
    char *plane; // Адрес начала строки с названием самолёта
    int relevant; // Флаг, который показывает, актуальный ли рейс: F = 0 or T = !F;
}flight;

/*Формирует рейс*/
flight FormAndInputFlight();
/*Ввод сколь угодно длинной (ага) строки*/
char* Input(FILE*);


int main()
{
    flight one; // Какой-то рейс
    one = FormAndInputFlight();
    printf("|%d|%s|%s|%s|%d|%c|\n", one.id, one.from, one.target, one.plane, one.quantityPassengers, (one.relevant?'Y':'N'));
    return 0;
}



flight FormAndInputFlight()
{
    flight one; // Структура, содержащая 1 рейс
    int buff; // Буффер для ввода чисел
    printf("Input id of flight: ");
    do
    {
        fflush(stdin);
        scanf("%d", &buff);
        if(buff <= 0)
            printf("Flight number cannot be negative. Input again: ");
    }while(buff <= 0);
    one.id = buff;

    printf("Where does the plane fly from?\n");
    one.from = Input(stdin);

    printf("Where does the plane fly to?\n");
    one.target = Input(stdin);

    printf("What is the name of the plane?\n");
    one.plane = Input(stdin);

    printf("Input number of passengers of flight: ");
    do
    {
        fflush(stdin);
        scanf("%d", &buff);
        if(buff < 0)
            printf("Number of passengers cannot be negative. Input again: ");
    }while(buff < 0);
    one.quantityPassengers = buff;

    printf("Is the flight up to date? (0 = NO and 1 = YES): ");
    scanf("%d", &(one.relevant));

    if(one.from == NULL || one.target == NULL || one.plane == NULL)
    {
        printf("Failed allocate flight successfully...\n");
    }
    return one;
}


/*
df - Указатель на структуру, с помощью которой будет производиться чтение строки
*/
char* Input(FILE *df)
{
    char *line = NULL; // Указатель на начало считываемой строки
    char *buff = NULL; // Указатель на начало строки (нужно для reallock)
    int i, // Счётчик для цикла
        max; // Максимальный размер строки
    int c, // Временный символ
        allOK; // Флаг успешности чтения или выхода
    line = (char*)malloc( (N+1) * sizeof(char));
    max = 1;
    i = 0;
    if(line != NULL)
        do
        {
            max += N;
            while(i < max && (c = fgetc(df)) != '\n')
            {
                line[i] = c;
                i++;
            }
            if(c == '\n')
                if (i > 0)
                {
                    line[i] = '\0';
                    line = (char*)realloc(line, (i+1)*sizeof(char)); // Тут и вылетает. realloc на уменьшение, чтобы line не занимал слишком много места (i на этом шаге = кол-ву символов в line)
                    allOK = 1;
                }
                else
                    allOK = 0;
            else
            {
                allOK = 0;
                buff = line;
                line = (char*)realloc(line, (i+N) * sizeof(char)); //Если есть ещё что считать, то нужно увеличить line, чтобы туда ещё вместилось. Это realloc на увеличение
                if(line == NULL)
                {
                    allOK = 1; //Чтобы выйти
                    printf(ERRORallocateLine);
                    free(buff);
                    buff = NULL;
                }
            }
        }while(!allOK);
    else
        printf(ERRORallocateLine);
    return line;
}
4
  • Обычные имплементации realloc в случае если новый размер меньше чем текущий, ничего не делают и просто возвращают указатель. Если Вы реально хотите это сделать - нужно ручками выделить нужное количество памяти и скопировать. Но не факт, что Вы получите какой-то существенный выиграш.
    – KoVadim
    8 апр 2020 в 15:21
  • Учту на будущее. Но почему вылетает то?
    – Bob
    8 апр 2020 в 15:29
  • И не стоит делать foo = realloc(foo, ...);.
    – 0andriy
    8 апр 2020 в 18:05
  • даже realloc на уменьшение?
    – Bob
    8 апр 2020 в 20:15

1 ответ 1

1

Везде после чтения чисел scanf("%d", &buff); нужно считать из буфера всё до перевода строки.

scanf("%d", &buff);
while ( getchar() != '\n') ;

При чтении Input() первый символ лишний '\n' мешает.

3
  • Спасибо. Можно узнать как вы пришли к такому выводу?
    – Bob
    8 апр 2020 в 17:20
  • Отладкой посмотрел, что первый символ равен '\n' и сразу увеличился буфер до 161. И тут дошло, что первые символы '\n' бывают после scanf. Так оно и оказалось.
    – AlexGlebe
    8 апр 2020 в 17:27
  • У меня ещё и неправильно работала функция Input()... Спасибо. Кто вы такие, люди со stackoverflow?
    – Bob
    8 апр 2020 в 17:35

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.