Возник вопрос, насколько безопасно так делать?

Ссылки в лямбде не протухнут на момент сработки таймера?
PS: вместо таймера можно подставить любой асинхронный объект, который триггернет лябду.

А как сейчас таймер до него добирается?

Никак не "перетечет", но не видно как позвать лямбду из др потока. Разве что сделать коннект с самоубийственным DirectConnection. А так (auto) прием будет в родном потоке, как и по таймеру

Др дело пользоваться этой конструкцией неудобно, коннект можно делать  "только здесь"

Ну как-то это "не очень просто/естественно " Если интересно "рухнет или нет" (из другого) - то да, рухнет, во всяком случае должно. Но написав waiter.exec Вы ясно показали что хотите делать все "легально". Ну и делайте, объявите waiter членом класса и добавьте слот c quit. Оформить ожидание "просто ф-цией" (возможно это цель?) - не стоит оно такого выноса мозга.

Зачем тут лямбду городить, можно же проще:

C++ (Qt)
class CWaitLoop : public QEventLoop
{
    Q_OBJECT
public:
    explicit CWaitLoop(QObject *parent = nullptr) : QEventLoop(parent) {}
    int wait(int msecs);
 
public slots:
    void exitSlot(int code) {
        QEventLoop::exit(code);
    }
};
 
int CWaitLoop::wait(int msecs){
    QTimer timer;
    timer.setSingleShot(true);
    timer.start(msecs);
    connect(&timer, &QTimer::timeout, this, &CWaitLoop::quit);
    return exec();
}
 
class AnyClass {
...
signals:
    void stopWaiting(int);
};
 
...
    AnyClass any;
    CWaitLoop waitLoop;
    connect(&any, &AnyClass::stopWaiting, &waitLoop, &CWaitLoop::exitSlot);
    int ret = waitLoop.wait(1000);
 

Этот ответ меня устраивает, а примечание про указатель развеивает все сомнения.

Мне не надо как проще, мне нужно понять, как это работает.