[FFmpeg-devel] [PATCH] lavfi/vf_srcnn: use avio_check instead of access

James Almer jamrial at gmail.com
Tue May 8 18:47:44 EEST 2018


On 5/8/2018 12:27 PM, Hendrik Leppkes wrote:
> The filter uses avio for file access already, and avio_check is
> portable.
> 
> Fixes trac #7192.

Remove the access dep in configure while at it.

LGTM is confirmed working.

> ---
>  libavfilter/vf_srcnn.c | 5 +----
>  1 file changed, 1 insertion(+), 4 deletions(-)
> 
> diff --git a/libavfilter/vf_srcnn.c b/libavfilter/vf_srcnn.c
> index dace2b99b5..edffebb278 100644
> --- a/libavfilter/vf_srcnn.c
> +++ b/libavfilter/vf_srcnn.c
> @@ -28,9 +28,6 @@
>  #include "formats.h"
>  #include "internal.h"
>  #include "libavutil/opt.h"
> -#if HAVE_UNISTD_H
> -#include <unistd.h>
> -#endif
>  #include "vf_srcnn.h"
>  #include "libavformat/avio.h"
>  
> @@ -145,7 +142,7 @@ static av_cold int init(AVFilterContext* context)
>          srcnn_context->conv3.size = 5;
>          CHECK_ALLOCATION(allocate_copy_conv_data(&srcnn_context->conv3, conv3_kernel, conv3_biases), )
>      }
> -    else if (access(srcnn_context->config_file_path, R_OK) != -1){
> +    else if (avio_check(srcnn_context->config_file_path, AVIO_FLAG_READ) > 0){
>          if (avio_open(&config_file_context, srcnn_context->config_file_path, AVIO_FLAG_READ) < 0){
>              av_log(context, AV_LOG_ERROR, "failed to open configuration file\n");
>              return AVERROR(EIO);
> 



More information about the ffmpeg-devel mailing list