diff options
author | Artem Dyomin <artem.dyomin@qt.io> | 2023-02-03 14:58:44 +0100 |
---|---|---|
committer | Qt Cherry-pick Bot <cherrypick_bot@qt-project.org> | 2023-02-03 15:17:10 +0000 |
commit | 0ed55c76bbca894fa7446150fdefe29b9ed8132f (patch) | |
tree | b7c77ba09137e9bc809c715cbf76edb9e84c2894 | |
parent | ba01d8928194fb2892da0f8b68d5cf992265824d (diff) |
Fix encoder UB if no audio inputs
The case is reproduced if QAudioInput is set to QMediaCaptureSession
but the platform doesn't have audio inputs.
The case is to be unit tested after implementation of
comprehensive mock of MediaDevices
Change-Id: I372c8c1260cb3fa0d70ea48a31c315bd9580cb06
Reviewed-by: Lars Knoll <lars@knoll.priv.no>
(cherry picked from commit c56de3e60cd3c86524b1767277a69d6e02e1350d)
Reviewed-by: Qt Cherry-pick Bot <cherrypick_bot@qt-project.org>
-rw-r--r-- | src/plugins/multimedia/ffmpeg/qffmpegmediarecorder.cpp | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/src/plugins/multimedia/ffmpeg/qffmpegmediarecorder.cpp b/src/plugins/multimedia/ffmpeg/qffmpegmediarecorder.cpp index 2fe1aeb6d..bf61989d5 100644 --- a/src/plugins/multimedia/ffmpeg/qffmpegmediarecorder.cpp +++ b/src/plugins/multimedia/ffmpeg/qffmpegmediarecorder.cpp @@ -77,8 +77,12 @@ void QFFmpegMediaRecorder::record(QMediaEncoderSettings &settings) connect(encoder, &QFFmpeg::Encoder::error, this, &QFFmpegMediaRecorder::handleSessionError); auto *audioInput = m_session->audioInput(); - if (audioInput) - encoder->addAudioInput(static_cast<QFFmpegAudioInput *>(audioInput)); + if (audioInput) { + if (audioInput->device.isNull()) + qWarning() << "Audio input device is null; cannot encode audio"; + else + encoder->addAudioInput(static_cast<QFFmpegAudioInput *>(audioInput)); + } auto *camera = m_session->camera(); if (camera) |