Re: [FFmpeg-devel] [PATCH v2 1/2] avformat/tee: Fix TeeSlave.bsfs pointer array size

2016-05-07 Thread Marton Balint 


On Thu, 5 May 2016, Nicolas George wrote:


Le quintidi 15 floréal, an CCXXIV, sebechlebsky...@gmail.com a écrit :

From: Jan Sebechlebsky 

TeeSlave.bsfs is array of pointers to AVBitStreamFilterContext,
so element size should be really size of a pointer, not size
of TeeSlave structure.

Signed-off-by: Jan Sebechlebsky 


Ok for me, of course.


Applied, thanks.

Marton
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH v2 1/2] avformat/tee: Fix TeeSlave.bsfs pointer array size

2016-05-05 Thread Nicolas George 
Le quintidi 15 floréal, an CCXXIV, sebechlebsky...@gmail.com a écrit :
> From: Jan Sebechlebsky 
> 
> TeeSlave.bsfs is array of pointers to AVBitStreamFilterContext,
> so element size should be really size of a pointer, not size
> of TeeSlave structure.
> 
> Signed-off-by: Jan Sebechlebsky 

Ok for me, of course.

Regards,

-- 
  Nicolas George


signature.asc
Description: Digital signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH v2 1/2] avformat/tee: Fix TeeSlave.bsfs pointer array size

2016-05-03 Thread sebechlebskyjan 
From: Jan Sebechlebsky 

TeeSlave.bsfs is array of pointers to AVBitStreamFilterContext,
so element size should be really size of a pointer, not size
of TeeSlave structure.

Signed-off-by: Jan Sebechlebsky 
---
 I've rewritten sizeof as suggested :)

 libavformat/tee.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavformat/tee.c b/libavformat/tee.c
index 499ef33..6d2ce53 100644
--- a/libavformat/tee.c
+++ b/libavformat/tee.c
@@ -324,7 +324,7 @@ static int open_slave(AVFormatContext *avf, char *slave, 
TeeSlave *tee_slave)
 }
 tee_slave->header_written = 1;
 
-tee_slave->bsfs = av_calloc(avf2->nb_streams, sizeof(TeeSlave));
+tee_slave->bsfs = av_calloc(avf2->nb_streams, sizeof(*tee_slave->bsfs));
 if (!tee_slave->bsfs) {
 ret = AVERROR(ENOMEM);
 goto end;
-- 
1.9.1

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel