[PATCH 6/8] rtp: Don't release memblock until we stop accessing underlying data

[tanuk@xxxxxx (Tanu Kaskinen)]

On Mon, 2016-02-29 at 15:46 +0530, arun at accosted.net wrote:
> From: Arun Raghavan <git at arunraghavan.net>
> 
> ---
> Â src/modules/rtp/rtp.c | 3 ++-
> Â 1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/src/modules/rtp/rtp.c b/src/modules/rtp/rtp.c
> index e372cbd..5a24a03 100644
> --- a/src/modules/rtp/rtp.c
> +++ b/src/modules/rtp/rtp.c
> @@ -256,7 +256,6 @@ int pa_rtp_recv(pa_rtp_context *c, pa_memchunk
> *chunk, pa_mempool *pool, uint32_
> Â Â Â Â Â m.msg_flags = 0;
> Â 
> Â Â Â Â Â r = recvmsg(c->fd, &m, 0);
> -Â Â Â Â pa_memblock_release(chunk->memblock);
> Â 
> Â Â Â Â Â if (r != size) {
> Â Â Â Â Â Â Â Â Â if (r < 0 && errno != EAGAIN && errno != EINTR)
> @@ -274,6 +273,8 @@ int pa_rtp_recv(pa_rtp_context *c, pa_memchunk
> *chunk, pa_mempool *pool, uint32_
> Â Â Â Â Â memcpy(rtp_tstamp, (uint8_t*) iov.iov_base + 4,
> sizeof(uint32_t));
> Â Â Â Â Â memcpy(&ssrc, (uint8_t*) iov.iov_base + 8, sizeof(uint32_t));
> Â 
> +Â Â Â Â pa_memblock_release(chunk->memblock);
> +
> Â Â Â Â Â header = ntohl(header);
> Â Â Â Â Â *rtp_tstamp = ntohl(*rtp_tstamp);
> Â Â Â Â Â ssrc = ntohl(c->ssrc);

Looks good!

--Â 
Tanu