BreakingY issues

Results 13 issues of


                                            BreakingY

作者是不维护了吗，项目还存在一些问题需要解决的，问题如下：

我看到的问题如下： 1、TaskScheduler::HandleTriggerEvent()应该加锁，多线程下有内存泄漏风险。 2、EpollTaskScheduler::~EpollTaskScheduler()也没有关闭epoll描述符，RtspServer项目中是关闭了的，作者没有同步好。 3、rtmp/src/xop/RtmpChunk.cpp RtmpChunk::ParseChunkHeader函数，rtmp_msg.stream_id = ReadUint24LE((char*)header.stream_id);这里不对stream_id是四个字节，应该使用ReadUint32LE解析，但是这里并不影响运行，因为RtmpConnection中并没有使用RtmpChunk中解析的，不过这也确实是一个bug。 4、RtmpConnection::HandleAudio(RtmpMessage& rtmp_msg)中play_cb_参数没传递正确，应该用uint8_t sound_format = (payload[0] >> 4) & 0x0f;，表示FLV中AAC定义的类型，FLV中H264是7 AAC是10 5、TimerQueue::HandleTimerEvent()：如果超时，定时器队列中的数据没有被删除，导致内存泄露 6、测试程序也有问题，RTMP使用FLV的tag data格式传输视频，NALU不应该包含起始码，不过我重新写了H264和AAC的推拉流测试程序，项目的推拉流功能是没问题的总体来说这是一个很好的rtmp项目，如果作者能维护下去，把存在的问题解决了，项目就完美了，作者还是牛逼的，写出了这么好的项目。关于作者的RtspServer也同样存在一些问题，我也提出来了，https://github.com/PHZ76/RtspServer/issues/92

obp_get_next_obu()

When reading the extension header in the obp_get_next_obu function, it should check if buf_size is less than 2 ``` if (buf_size < 2) { snprintf(err->error, err->size, "Buffer is too small...

不加nms，有很多重叠框

使用TensorRT不加nms为什么有很多重叠框呢，论文不是说不需要nms吗，是哪里出了问题。问题复现项目地址：https://github.com/BreakingY/D-FINE_onnx_tensorrt