# V2582. MISRA. Block of memory should only be freed if it was allocated by a Standard Library function.

Данное диагностическое правило основано на руководстве MISRA (Motor Industry Software Reliability Association) по разработке программного обеспечения.

Это правило актуально только для C. Память, динамически выделенная при помощи функций 'malloc', 'calloc', 'realloc', должна быть освобождена функцией 'free'. Блок памяти, освобождённый при помощи функции 'free', нельзя передавать в 'free' повторно. Такие действия ведут к неопределённому поведению программы.

Рассмотрим первый пример:

    void foo()
    {
      int arr[50];
      // ....
      free(arr);
    }

Здесь анализатор обнаружил ошибочный код, в котором осуществляется попытка удаления массива через функцию 'free'. Однако память под массив выделена на стеке и будет освобождена автоматически при выходе из функции.

Рассмотрим ещё пример:

    void foo()
    {
      float *p1 = (float *)malloc(N * sizeof(float));
      float *p2 = (float *)malloc(K * sizeof(float));
      // ....
      free(p1);
      free(p1);
    }

В коде имеется опечатка, из-за которой дважды освобождается одна и та же область памяти по указателю 'p1'. В результате, во-первых, возникает утечка памяти, так как не освобождается буфер, адрес которого хранится в переменной 'p2'. Во-вторых, из-за повторного освобождения одного и того же буфера возникает неопределённое поведение.

Корректный вариант:

    void foo()
    {
      float *p1 = (float *)malloc(N * sizeof(float));
      float *p2 = (float *)malloc(K * sizeof(float));
      // ....
      free(p1);
      free(p2);
    }

Данная диагностика классифицируется как:

MISRA-C-2012-22.2 MISRA-C-2023-22.2