# V509. Exceptions raised inside noexcept functions must be wrapped in a try..catch block.

Если в программе возникает исключение, начинается свертывание стека, в ходе которого объекты разрушаются путем вызова деструкторов. Если деструктор объекта, разрушаемого при свертывании стека, бросает еще одно исключение и это исключение покидает деструктор, библиотека C++ немедленно аварийно завершает программу, вызывая функцию terminate(). Из этого следует, что деструкторы никогда не должны распространять исключения. Исключение, брошенное внутри деструктора, должно быть обработано внутри того же деструктора.

Анализатор обнаружил деструктор, содержащий оператор throw вне блока try..catch. Пример:

    LocalStorage::~LocalStorage()
    {
      ...
      if (!FooFree(m_index))
        throw Err("FooFree", GetLastError());
      ...
    }

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

    LocalStorage::~LocalStorage()
    {
      try {
        ...
        if (!FooFree(m_index))
          throw Err("FooFree", GetLastError());
        ...
      }
      catch (...)
      {
        assert(false);
      }
    }

Так же исключения могут возникать при вызове оператора 'new'. При невозможности выделения памяти будет сгенерировано исключение 'bad_alloc'. Пример:

    A::~A()
    {
      ...
      int *localPointer = new int[MAX_SIZE];
      ...
    }

Появление исключения возможно при использовании dynamic_cast<Type> при работе с ссылками. При невозможности приведения типов будет сгенерировано исключение 'bad_cast'. Пример:

    B::~B()
    {
      ...
      UserType &type = dynamic_cast<UserType&>(baseType);
      ...
    }

Для исправления данных ошибок следует переписать код таким образом, чтобы 'new' или 'dynamic_cast' были помещены в блок 'try{...}'.

Начиная с C++11 функции могут быть размечены как 'noexcept'. Если такие функции выбрасывают исключение, это приводит к аварийному завершению программы. Анализатор обнаруживает вызовы, которые потенциально могут выбросить исключение, в 'noexcept' функциях. Пример:

    int noexceptWithNew() noexcept
    {
      return *(new int{42});
    }

Анализатор выдаст предупреждение, так как оператор 'new' может выбросить исключение. Вызов 'new' в этом случае нужно обернуть в блок 'try..catch'.

Анализатор также обнаруживает вызовы функций, не размеченных как 'noexcept', из деструкторов и 'noexcept' функций. Такие вызовы потенциально опасны, так как они могут приводить к исключениям. Рассмотрим пример:

    int allocate_memory()
    {
      return *(new int{ 42 });
    }

    int noexceptFunc() noexcept
    {
      return allocate_memory();
    }

Анализатор выдаст предупреждение на строке с вызовом функции 'allocate_memory'.

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

Дополнительные материалы по данной теме:

- Bjarne Stroustrup's C++ Style and Technique FAQ. Can I throw an exception from a constructor? From a destructor? http://www.stroustrup.com/bs_faq2.html
- Throwing destructors. http://www.kolpackov.net/projects/c++/eh/dtor-1.xhtml

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

CWE-248 CWE-703 CERT-DCL57-CPP

Взгляните на примеры ошибок, обнаруженных с помощью диагностики V509.