diff mbox

[FFmpeg-devel,1/3] avfilter/vf_sr: fix read out of bounds

Message ID 20180913074952.2873-1-quinkblack@foxmail.com
State Superseded
Headers show

Commit Message

Zhao Zhili Sept. 13, 2018, 7:49 a.m. UTC
---
 libavfilter/vf_sr.c | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

Comments

Zhao Zhili Sept. 13, 2018, 8:01 a.m. UTC | #1
Please drop this one. I have sent a version use const int array.

On 2018年09月13日 15:49, Zhao Zhili wrote:
> ---
>   libavfilter/vf_sr.c | 9 ++++++---
>   1 file changed, 6 insertions(+), 3 deletions(-)
>
> diff --git a/libavfilter/vf_sr.c b/libavfilter/vf_sr.c
> index 5ad1baa..26cacde 100644
> --- a/libavfilter/vf_sr.c
> +++ b/libavfilter/vf_sr.c
> @@ -239,7 +239,8 @@ static int filter_frame(AVFilterLink *inlink, AVFrame *in)
>                     0, sr_context->sws_slice_h, out->data, out->linesize);
>   
>           sws_scale(sr_context->sws_contexts[1], (const uint8_t **)out->data, out->linesize,
> -                  0, out->height, (uint8_t * const*)(&sr_context->input.data), &sr_context->sws_input_linesize);
> +                  0, out->height, (uint8_t * const*)(&sr_context->input.data),
> +                  (int [4]){sr_context->sws_input_linesize, 0, 0, 0});
>           break;
>       case ESPCN:
>           if (sr_context->sws_contexts[0]){
> @@ -250,7 +251,8 @@ static int filter_frame(AVFilterLink *inlink, AVFrame *in)
>           }
>   
>           sws_scale(sr_context->sws_contexts[1], (const uint8_t **)in->data, in->linesize,
> -                  0, in->height, (uint8_t * const*)(&sr_context->input.data), &sr_context->sws_input_linesize);
> +                  0, in->height, (uint8_t * const*)(&sr_context->input.data),
> +                  (int [4]){sr_context->sws_input_linesize, 0, 0, 0});
>       }
>       av_frame_free(&in);
>   
> @@ -260,7 +262,8 @@ static int filter_frame(AVFilterLink *inlink, AVFrame *in)
>           return AVERROR(EIO);
>       }
>   
> -    sws_scale(sr_context->sws_contexts[2], (const uint8_t **)(&sr_context->output.data), &sr_context->sws_output_linesize,
> +    sws_scale(sr_context->sws_contexts[2], (const uint8_t **)(&sr_context->output.data),
> +              (int [4]){sr_context->sws_output_linesize, 0, 0, 0},
>                 0, out->height, (uint8_t * const*)out->data, out->linesize);
>   
>       return ff_filter_frame(outlink, out);
diff mbox

Patch

diff --git a/libavfilter/vf_sr.c b/libavfilter/vf_sr.c
index 5ad1baa..26cacde 100644
--- a/libavfilter/vf_sr.c
+++ b/libavfilter/vf_sr.c
@@ -239,7 +239,8 @@  static int filter_frame(AVFilterLink *inlink, AVFrame *in)
                   0, sr_context->sws_slice_h, out->data, out->linesize);
 
         sws_scale(sr_context->sws_contexts[1], (const uint8_t **)out->data, out->linesize,
-                  0, out->height, (uint8_t * const*)(&sr_context->input.data), &sr_context->sws_input_linesize);
+                  0, out->height, (uint8_t * const*)(&sr_context->input.data),
+                  (int [4]){sr_context->sws_input_linesize, 0, 0, 0});
         break;
     case ESPCN:
         if (sr_context->sws_contexts[0]){
@@ -250,7 +251,8 @@  static int filter_frame(AVFilterLink *inlink, AVFrame *in)
         }
 
         sws_scale(sr_context->sws_contexts[1], (const uint8_t **)in->data, in->linesize,
-                  0, in->height, (uint8_t * const*)(&sr_context->input.data), &sr_context->sws_input_linesize);
+                  0, in->height, (uint8_t * const*)(&sr_context->input.data),
+                  (int [4]){sr_context->sws_input_linesize, 0, 0, 0});
     }
     av_frame_free(&in);
 
@@ -260,7 +262,8 @@  static int filter_frame(AVFilterLink *inlink, AVFrame *in)
         return AVERROR(EIO);
     }
 
-    sws_scale(sr_context->sws_contexts[2], (const uint8_t **)(&sr_context->output.data), &sr_context->sws_output_linesize,
+    sws_scale(sr_context->sws_contexts[2], (const uint8_t **)(&sr_context->output.data),
+              (int [4]){sr_context->sws_output_linesize, 0, 0, 0},
               0, out->height, (uint8_t * const*)out->data, out->linesize);
 
     return ff_filter_frame(outlink, out);