fix(audio): add thread safety checks and debug logging for audio proc…

[add-uos]

…essing

add thread safety checks and debug logging for audio processing

log: add thread safety checks and debug logging for audio processing
bug: https://pms.uniontech.com/bug-view-340143.html

[deepin-ci-robot]

deepin pr auto review

我来对这段代码进行审查，主要关注以下几个方面：

1. 代码逻辑和线程安全
2. 错误处理
3. 资源管理
4. 性能优化
5. 日志记录

改进建议：

1. 线程状态检查优化：

// 在 writMixAudio() 中
while (bWriteMix() && m_bMix) {
    // 建议使用原子变量或互斥锁来保护这些状态变量
    if (!bWriteMix() || !m_bMix) {
        break; // 这个检查是冗余的，因为while条件已经包含
    }
    m_context->m_recordAdmin->m_pOutputStream->writeMixAudio();
}

2. 资源管理改进：

// 在 readMicToMixAudioPacket() 中
AVFrame *inputFrame = avlibInterface::m_av_frame_alloc();
if (!inputFrame) {
    qDebug() << "readMicToMixAudioPacket: Failed to allocate inputFrame";
    // 建议在这里设置错误状态，而不是直接返回
    setbRunThread(false);
    return AVERROR(ENOMEM);
}
// 使用 RAII 或智能指针来管理资源

3. 错误处理增强：

// 在 writeMicToMixAudioFrame() 中
if (!m_pMicCodecContext || !m_convertedMicSamples || !isWriteFrame()) {
    qDebug() << "writeMicToMixAudioFrame: Resource check failed";
    // 建议返回具体的错误码，而不是统一的-1
    return AVERROR(EINVAL); // 或其他适当的错误码
}

4. 性能优化建议：

• 减少不必要的日志输出，特别是在高频循环中
• 考虑使用更高效的线程同步机制
• 对于音频处理，可以考虑批量处理帧而不是逐帧处理

5. 日志记录改进：

// 使用统一的日志格式和日志级别
qDebug() << "[DEBUG] FunctionName: Message";
qWarning() << "[WARN] FunctionName: Message";
qCritical() << "[ERROR] FunctionName: Message";

6. 代码结构优化：

• 考虑将状态检查逻辑抽取为单独的函数
• 使用枚举代替魔法数字
• 增加函数文档注释

7. 安全性改进：

• 对外部输入进行验证
• 检查指针有效性后再使用
• 使用安全的字符串操作函数

8. 具体修改建议：

// 添加资源清理的RAII包装器
class AVFrameGuard {
    AVFrame* frame;
public:
    AVFrameGuard() : frame(avlibInterface::m_av_frame_alloc()) {}
    ~AVFrameGuard() { if(frame) avlibInterface::m_av_frame_free(&frame); }
    AVFrame* get() { return frame; }
};

// 使用示例
void CAVInputStream::readMicToMixAudioPacket() {
    AVFrameGuard frameGuard;
    if (!frameGuard.get()) {
        return AVERROR(ENOMEM);
    }
    // 使用 frameGuard.get() ...
}

这些改进将使代码更加健壮、安全和可维护。建议逐步实施这些改进，并在每个阶段进行充分的测试。

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: add-uos, max-lvs

The full list of commands accepted by this bot can be found here.

Details

Needs approval from an approver in each of these files:

• debian/deepin/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[add-uos]

/merge