Commit 607b5458 authored by Rémi Denis-Courmont's avatar Rémi Denis-Courmont

- Fix error handling

 - Fix a warning
parent 2004f95f
...@@ -326,16 +326,17 @@ static block_t *BlockUDP( access_t *p_access ) ...@@ -326,16 +326,17 @@ static block_t *BlockUDP( access_t *p_access )
{ {
access_sys_t *p_sys = p_access->p_sys; access_sys_t *p_sys = p_access->p_sys;
block_t *p_block; block_t *p_block;
ssize_t len;
if( p_access->info.b_eof ) if( p_access->info.b_eof )
return NULL; return NULL;
/* Read data */ /* Read data */
p_block = block_New( p_access, MTU ); p_block = block_New( p_access, MTU );
p_block->i_buffer = net_Read( p_access, p_sys->fd, NULL, len = net_Read( p_access, p_sys->fd, NULL,
p_block->p_buffer, MTU, VLC_FALSE ); p_block->p_buffer, MTU, VLC_FALSE );
if( ( p_block->i_buffer < 0 ) if( ( len < 0 )
|| ( p_sys->b_comedia && ( p_block->i_buffer == 0 ) ) ) || ( p_sys->b_comedia && ( len == 0 ) ) )
{ {
if( p_sys->b_comedia ) if( p_sys->b_comedia )
{ {
...@@ -346,7 +347,7 @@ static block_t *BlockUDP( access_t *p_access ) ...@@ -346,7 +347,7 @@ static block_t *BlockUDP( access_t *p_access )
return NULL; return NULL;
} }
return block_Realloc( p_block, 0, p_block->i_buffer ); return block_Realloc( p_block, 0, p_block->i_buffer = len );
} }
/***************************************************************************** /*****************************************************************************
...@@ -395,7 +396,7 @@ static block_t *BlockTCP( access_t *p_access ) ...@@ -395,7 +396,7 @@ static block_t *BlockTCP( access_t *p_access )
p_block->i_buffer += i_read; p_block->i_buffer += i_read;
} }
if( p_block->i_buffer < (2 + framelen) ) if( p_block->i_buffer < (2u + framelen) )
return NULL; // incomplete frame return NULL; // incomplete frame
/* Hide framing from RTP layer */ /* Hide framing from RTP layer */
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment