Junlong Zou

Results 19 comments of Junlong Zou

`uv_udp_try_send` accept several buffers at a time. Maybe we can save header and payload in different buffers, so that we don't need to shift payload.

Well, we have to encrypt payload before sending, and currently libsrtp don't support read header and payload from different buffer. But in `SrtpSession::EncryptRtp`, we will copy original rtp packet to...

We can also do this when sending rtx packet.

In webrtc, they use a `Copy-On-Write` buffer to store rtp packet, it can save memory, however i think it can not reduce the number of memcopy.

> What is this fixing exactly? I've never seen those tests failing due to memory problems. When i ran tests under windows, it says buffer is overflow. Maybe u can...

``` (mediasoup_buildvenv) [root@784dfa00-aa18-11ee-bad9-a4bf015565d5 worker]# ./out/Debug/mediasoup-worker-test-asan Randomness seeded to: 446808779 ================================================================= ==2099730==ERROR: AddressSanitizer: stack-buffer-overflow on address 0x7fff7ac12c80 at pc 0x7f231efafe7e bp 0x7fff7abf9940 sp 0x7fff7abf90f0 WRITE of size 16 at 0x7fff7ac12c80 thread...

By the way, when i ran mediasoup_test_asan release version, it show: ``` AddressSanitizer:DEADLYSIGNAL ================================================================= ==2092397==ERROR: AddressSanitizer: SEGV on unknown address 0x000000000000 (pc 0x0000004e4d91 bp 0x00000000001f sp 0x7ffe438c49e0 T0) ==2092397==The signal...

This PR fix the same problem as https://github.com/versatica/mediasoup/pull/1120

It seems this project is dead. Try typora.io, it's free and also supports math and diagrams.