[FFmpeg-cvslog] lavfi/pad: avoid buffer overflow in buffer_needs_copy()

Xi Wang git at videolan.org
Sun Mar 10 10:56:12 CET 2013


ffmpeg | branch: master | Xi Wang <xi.wang at gmail.com> | Sat Mar  9 22:39:32 2013 -0500| [969e8d35b5b4c457b33df524ca95cd548a404599] | committer: Stefano Sabatini

lavfi/pad: avoid buffer overflow in buffer_needs_copy()

Replace & with short-circuit &&.

Signed-off-by: Xi Wang <xi.wang at gmail.com>
Signed-off-by: Stefano Sabatini <stefasab at gmail.com>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=969e8d35b5b4c457b33df524ca95cd548a404599
---

 libavfilter/vf_pad.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavfilter/vf_pad.c b/libavfilter/vf_pad.c
index 86fd683..3f4a4f0 100644
--- a/libavfilter/vf_pad.c
+++ b/libavfilter/vf_pad.c
@@ -300,7 +300,7 @@ static int buffer_needs_copy(PadContext *s, AVFrame *frame, AVBufferRef *buf)
             return 1;
 
 #define SIGN(x) ((x) > 0 ? 1 : -1)
-        for (j = 0; j < FF_ARRAY_ELEMS(planes) & planes[j] >= 0; j++) {
+        for (j = 0; j < FF_ARRAY_ELEMS(planes) && planes[j] >= 0; j++) {
             int hsub1 = s->draw.hsub[planes[j]];
             uint8_t *start1 = frame->data[planes[j]];
             uint8_t *end1   = start1 + (frame->height >> hsub1) *



More information about the ffmpeg-cvslog mailing list