OpenRTMFP/Cumulus Primer（19）独立使用CumulusLib时的线程安全Bug

OpenRTMFP/Cumulus Primer（19）独立使用CumulusLib时的线程安全Bug

• 作者：柳大·Poechant（钟超）
• 邮箱：zhongchao.ustc#gmail.com（# -> @）
• 博客：Blog.CSDN.net/Poechant
• 日期：June 7^nd, 2012

OpenRTMFP/Cumulus 提供了 CumulusLib 可以供其他 RTMFP 应用使用，而不局限于 CumulusServer。

一般来说，Thread A 会准备好要 push 的消息，然后 Thread A 向消息队列 push 消息。

但是 CumulusLib 中实现的，是 Thread A 向消息队列 push 消息，然后根据这个消息在队列中的指针，再向消息内填写字段。并期望如下：

由于在 CumulusServer 中，一个 Client 只在一个线程内被操作，相应的 FlowWriter 也不会出现跨线程的问题。但是如果单独使用 CumulusLib，如果出现线程通信，并且共享 FlowWriter 的话，就会共享消息队列，此时可能出现这种情况。

这就导致了很严重的错误，会使得进程崩溃。修正的方式，可以是将消息完全准备好之后，再放入队列，如下：

/*
 * author:  michael
 * date:    June 6th, 2012
 * type:    add
 */
MessageBuffered* FlowWriter::createAMFMessage(const std::string& name)

    // signature.empty() means that we are on the flowWriter of FlowNull
    if (!(_closed || signature.empty() || _band.failed())) {
        MessageBuffered* pMessage = new MessageBuffered();
        MessageBuffered& message(*pMessage);
        writeResponseHeader(message.rawWriter,name,0);
        return pMessage;
    }

    MessageBuffered& message(_MessageNull);
    writeResponseHeader(message.rawWriter,name,0);
    return NULL;
}

然后再调用时最后再增加 push 操作：

/*
 * author:  michael
 * date:    June 6th, 2012
 * type:    add
 */
void FlowWriter::pushAMFMessage(MessageBuffered* pMessage) {
    if (pMessage != NULL) {
        _messages.push_back(pMessage);
    }
}

这样就使得消息的数据被写完了，才被放入队列中，如下：

不过如果考虑线程安全，多个线程对同一个消息队列进行操作时，就要加锁：

/*
 * author:  michael
 * date:    June 6th, 2012
 * type:    add
 */
void FlowWriter::pushAMFMessage(MessageBuffered* pMessage) {
    if (pMessage != NULL) {
        Poco::Mutex::ScopedLock lock(msgQueueMutex);
        _messages.push_back(pMessage);
    }
}

这样就基本解决了这个线程安全问题。

另外，使用 CumulusLib 要遵循 GPL 协议，一定不要忘记。

-

转载请注明来自柳大·Poechant（钟超）的CSDN博客：Blog.CSDN.net/Poechant

-