Skip to content
  • David Faure's avatar
    Fix memory leak due to getTestMonitor() · 66b24c88
    David Faure authored
    Summary:
    asan build said the Monitor was leaked, and it was indeed.
    Fixed using std::unique_ptr.
    
    Test Plan:
    cmake -DCMAKE_C_COMPILER=clang -DCMAKE_CXX_COMPILER=clang++ -DECM_ENABLE_SANITIZERS="address" -DCMAKE_CXX_FLAGS='-stdlib=libc++' -DCMAKE_SHARED_LINKER_FLAGS='-stdlib=libc++' -DCMAKE_EXE_LINKER_FLAGS='-stdlib=libc++' .. && make && ctest
    (with an asan build of Qt)
    
    Reviewers: dvratil
    
    Reviewed By: dvratil
    
    Subscribers: kde-pim
    
    Differential Revision: https://phabricator.kde.org/D21093
    66b24c88