Почему free() срабатывает только один раз?

Question

[NaName]

Есть такой код на Си:

SIZE_J = 100500 - максимально возможный размер массива данных

void getDataFromFile(char *file_name, double *edmaxlat, double *edmaxlon, double latitude[], double longitude[], int *size) {
    int nc_file_id;
    int retval;
    int GEO_lat_id, GEO_lon_id;
    double *latitude_buffer = (double*)malloc(sizeof(double)*SIZE_J);
    double *longitude_buffer = (double*)malloc(sizeof(double)*SIZE_J);

    открыл файл
    if ((retval = nc_open(file_name, NC_NOWRITE, &nc_file_id)))
        ERR(retval);

    узнал ID переменных с нужными именами
    if ((retval = nc_inq_varid(nc_file_id, "GEO_lat", &GEO_lat_id)))
        ERR(retval);
    if ((retval = nc_inq_varid(nc_file_id, "GEO_lon", &GEO_lon_id)))
        ERR(retval);

    читаю переменные из файла по их ID
    if ((retval = nc_get_var_double(nc_file_id, GEO_lat_id, &latitude_buffer[0])))
        ERR(retval);
    if ((retval = nc_get_var_double(nc_file_id, GEO_lon_id, &longitude_buffer[0])))
        ERR(retval);

    for (int j = 0; j < SIZE_J; j++) {
        эти две переменные не нужны, использовал для принтов в цикле
        latitude[j] = latitude_buffer[j];
        longitude[j] = longitude_buffer[j];

        запоминаю размер массива. используется в других функциях
        if (height[j] <= 0.000000000001) {
            *size = j;
            break;
        }

    }
    free(latitude_buffer);
    free(longitude_buffer);

При работе этой функции в цикле без free(), latitude_buffer и longitude_buffer запоминают значения предыдущих итераций. Т.е. например в первой итерации было 10 элементов в буфере, во второй итерации 5 элементов в буфере (входные данные), тогда во второй итерации в переменной latitude будет 10 элементов. Не знаю почему так получается поэтому попробовал использовать free(). Но с free() тоже проблема. Если вызывать free() только один раз, например закомментировать второй вызов, то всё ок. Т.е. latitude всегда хранит адекватные значения, но longitude попрежнему переполняется (?). Если закомментировать первый вызов то тоже все ок. longitude хранит адекватные значения а latitude переполняется (?). Проблема в том что если оставить оба вызова, как в примере кода выше, то выполняется только первый free(), как-будто второй вызов закомментирован. Почему так происходит? Как сделать чтобы чистились оба буфера, или как сделать чтобы оба буфера не надо было чистить, но можно было сделать их переменной размерности? Да, в Си я очень плохо разбираюсь.

Comments

[Арсений Ефремов]

latitude_buffer и longitude_buffer запоминают значения предыдущих итераций.

Что? Из куска кода видно только, что у вас оба buffer'a заполнены мусором и нигде в них ничего не записывается.
И вы уверены, что у вас longitude и latitude, переданные в параметрах, всегда размера >= SIZE_J ?

[NaName]

Арсений Ефремов: Извините была ошибка в коде примера. Точно знаю, что количество элементов longitude и latitude всегда <= SIZE_J.

[Арсений Ефремов]

NaName: а теперь у вас значения передаются только в первый элемент массивов *buffer.
Во-первых, попробуйте вызывать nc_get_var_double в теле цикла для каждого элемента буфера (а не снаружи). Во-вторых, у вас буфер вообще не нужен, т.к. он просто заполняется, а потом сливается в другой буфер. Можете использовать для этого сразу longitude и latitiude.
В-третьих, неясно, что такое height, т.к. он нигде не объявлен. Это глобальная переменная? В любом случае, если у вас массивы <= SIZE_J, причём всегда, то практически гарантированно у вас будет переполнение. (А если условие не выполнится до момента, когда j станет больше размеров longitude и latitiude?)
Короче, вам нужно тщательно всё переписать.

Answer 1

[Денис Загаевский]

В каком смысле "буферы переполняются"? У вас буферы - это просто указатели на область памяти, в целом вы можете хоть до потери пульса по ним итерироваться.
И потом, malloc отдаёт вам неинициализированную память, то есть там просто мусор. Не вижу, чтобы вы как-либо инициализирвали их. Используйте memset, например.

дальше, free это правильно. Надо делать free всегда, когда вы делаете malloc.

Comments

[NaName]

Переполняется, т.е. запоминает значения мусора из предыдущей итерации. Для каждой итерации использую переменную size чтобы отделять мусор от немусора и все работает, т.е. питоний скрипт и этот код дают одинаковый результат, если не запускать этот код в цикле. Как только запускаю в цикле получается какаято фигня, какбудто значения предыдущих итерций цикла влияют на текущие значения в массивах. Код в вопросе исправил.

[Денис Загаевский]

NaName: Ещё раз. Ваш код не делает ничего полезного, поскольку вы пользуетесь неинициализированной памятью. Вы не записали никаких значений, но читаете оттуда. исправьте эту ошибку и дальше можно будет говорить предметно. Могу сказать точно, что в том виде, в котором вы код предоставили, локальные переменные не могут сохранять никаких значений.

[NaName]

Денис Загаевский: Исправил

Answer 2

[Mercury13]

Переполняется longitude, говорите? Ну убедитесь, что он размера SIZE_J. Не этой функции, правда, дело.
Кроме того, вы массивы latitude[] и longitude[] заполняете случайным мусором, появившимся в результате malloc.

Comments

[NaName]

Извините была ошибка в коде примера