[vlc-devel] [PATCH] dsm: add missing I/O error handling

Thomas Guillem thomas at gllm.fr
Mon Feb 24 09:46:51 CET 2020



On Fri, Feb 21, 2020, at 21:04, Janboe Ye wrote:
> From 7bba85e4f3a01b5afaa624e52c7534580f840f77 Mon Sep 17 00:00:00 2001
> From: Janboe Ye <janboe.ye at outlook.com>
> Date: Sat, 22 Feb 2020 04:00:39 +0800
> Subject: [PATCH] dsm: add missing I/O error handling
> 
> Signed-off-by: Janboe Ye <janboe.ye at outlook.com>
> ---
>  modules/access/dsm/access.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/modules/access/dsm/access.c b/modules/access/dsm/access.c
> index 69f61b9860..795872092e 100644
> --- a/modules/access/dsm/access.c
> +++ b/modules/access/dsm/access.c
> @@ -523,6 +523,8 @@ static ssize_t Read( stream_t *p_access, void 
> *p_buffer, size_t i_len )
>      if( i_read < 0 )
>      {
>          msg_Err( p_access, "read failed" );
> +        if (errno != EINTR && errno != EAGAIN)

It is not the contrary ?
EINTR or EAGAIN should not be treated as error, right ?

> +            return 0;
>          return -1;
>      }
>  
> -- 
> 2.23.0
> 
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel


More information about the vlc-devel mailing list