CRTP с unique_ptr вызывает segfault - PullRequest
Новая
       7

CRTP с unique_ptr вызывает segfault

0 голосов
/ 28 августа 2018

Я использую шаблон проектирования CRTP для реализации механизма ведения журнала для моего проекта. Базовый класс CRTP выглядит так: 

#include <fstream>
#include <memory>
#include <mutex>
#include <iostream>
#include <sstream>

template <typename LogPolicy>
class Logger
{
  public:
    template <typename... Args>
    void operator()(Args... args)
    {
        loggingMutex.lock();
        putTime();
        print_impl(args...);
    }

    void setMaxLogFileSize(unsigned long maxLogFileSizeArg)
    {
        //if (dynamic_cast<FileLogPolicy *>(policy.get()))
        //    policy->setMaxLogFileSize(maxLogFileSizeArg);
    }

    ~Logger()
    {
        print_impl(END_OF_LOGGING);
    }

  protected:
    std::stringstream buffer;
    std::mutex loggingMutex;
    std::string d_time;
  private:
    static constexpr auto END_OF_LOGGING = "***END OF LOGGING***";

    void putTime()
    {
        time_t raw_time;
        time(&raw_time);
        std::string localTime = ctime(&raw_time);
        localTime.erase(std::remove(localTime.begin(), localTime.end(), '\n'), localTime.end());
        buffer << localTime;
    }

    template <typename First, typename... Rest>
    void print_impl(First first, Rest... rest)
    {
        buffer << " " << first;
        print_impl(rest...);
    }

    void print_impl()
    {
        static_cast<LogPolicy*>(this)->write(buffer.str());
        buffer.str("");
    }
};

Один из конкретных классов ведения журнала - это запись в файл, которая выглядит следующим образом: 

#include "Logger.hpp"

class FileLogPolicy : public Logger<FileLogPolicy>
{
  public:
    FileLogPolicy(std::string fileName) : logFile(new std::ofstream)
    {
        logFile->open(fileName, std::ofstream::out | std::ofstream::binary);
        if (logFile->is_open())
        {
            std::cout << "Opening stream with addr " << (logFile.get()) << std::endl;
        }
    }

    void write(const std::string content)
    {

        std::cout << "Writing stream with addr " << (logFile.get()) << std::endl;
        (*logFile) << " " << content << std::endl;
        loggingMutex.unlock();
    }

    virtual ~FileLogPolicy()
    {
    }

  private:
    std::unique_ptr<std::ofstream> logFile; //Pointer to logging stream
    static const char *const S_FILE_NAME;   //File name used to store logging
    size_t d_maxLogFileSize;         //File max size used to store logging
};

В основном я создаю объект класса политики и хотел бы регистрировать материал в зависимости от выбранной политики. Так, например, я создаю регистратор, как это: 

FileLogPolicy log("log.txt");

В этом случае следует использовать Logger для сохранения журналов в файл, вызвав static_cast<LogPolicy*>(this)->write(buffer.str()). Очевидно, что вызывающая функция записи работает нормально, но объект потока изменяется на ноль. Как это возможно, если деструктор FileLogPolicy еще не был вызван? Когда я изменяю logFile на обычный указатель, все работает хорошо. Я не понимаю, в чем разница.

1 Ответ

0 голосов
/ 28 августа 2018 
~Logger()
{
    print_impl(END_OF_LOGGING);
}

этот код запускается после уничтожения класса спуска. 

void print_impl()
{
    static_cast<LogPolicy*>(this)->write(buffer.str());
    buffer.str("");
}

затем преобразует this в указатель на класс, которого this больше нет.

Уникальный ptr пропал, и даже доступ к члену - UB.

...