diff options
| author | Michael Niedermayer <michael@niedermayer.cc> | 2016-01-17 18:57:01 +0100 |
|---|---|---|
| committer | Michael Niedermayer <michael@niedermayer.cc> | 2016-01-28 15:53:53 +0100 |
| commit | 6897859b5acf6b4bf85f5cb735ffd6e14f55448d (patch) | |
| tree | 165af0c5bff0c3aefdfd5d63ccfb6d3645bd8ce4 | |
| parent | f121ed611ef3141995605eb48714478d9d8af908 (diff) | |
| download | ffmpeg-6897859b5acf6b4bf85f5cb735ffd6e14f55448d.tar.gz | |
swscale/swscale: Add some sanity checks for srcSlice* parameters
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
(cherry picked from commit 321e85e1769ca1fc1567025ae264760790ee7fc9)
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
| -rw-r--r-- | libswscale/swscale.c | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/libswscale/swscale.c b/libswscale/swscale.c index 722909a53e..c766b042fd 100644 --- a/libswscale/swscale.c +++ b/libswscale/swscale.c @@ -1044,12 +1044,20 @@ int attribute_align_arg sws_scale(struct SwsContext *c, const uint8_t *src2[4]; uint8_t *dst2[4]; uint8_t *rgb0_tmp = NULL; + int macro_height = isBayer(c->srcFormat) ? 2 : (1 << c->chrSrcVSubSample); if (!srcStride || !dstStride || !dst || !srcSlice) { av_log(c, AV_LOG_ERROR, "One of the input parameters to sws_scale() is NULL, please check the calling code\n"); return 0; } + if ((srcSliceY & (macro_height-1)) || + ((srcSliceH& (macro_height-1)) && srcSliceY + srcSliceH != c->srcH) || + srcSliceY + srcSliceH > c->srcH) { + av_log(c, AV_LOG_ERROR, "Slice parameters %d, %d are invalid\n", srcSliceY, srcSliceH); + return AVERROR(EINVAL); + } + if (c->gamma_flag && c->cascaded_context[0]) { |