Re: [FFmpeg-devel] [PATCH 1/2] avformat/dvenc: return error code of dv_assemble_frame()

2020-06-19 Thread lance . lmwang
On Fri, Jun 12, 2020 at 10:31:56PM +0800, lance.lmw...@gmail.com wrote:
> From: Limin Wang 
> 
> Signed-off-by: Limin Wang 
> ---
>  libavformat/dvenc.c | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/libavformat/dvenc.c b/libavformat/dvenc.c
> index c71e532..b04d604 100644
> --- a/libavformat/dvenc.c
> +++ b/libavformat/dvenc.c
> @@ -406,9 +406,10 @@ static int dv_write_packet(struct AVFormatContext *s, 
> AVPacket *pkt)
>  
>  fsize = dv_assemble_frame(s, s->priv_data, s->streams[pkt->stream_index],
>pkt->data, pkt->size, &frame);
> -if (fsize > 0) {
> -avio_write(s->pb, frame, fsize);
> +if (fsize < 0) {
> +return fsize;
>  }
> +avio_write(s->pb, frame, fsize);
>  return 0;
>  }
>  
> -- 
> 1.8.3.1
> 

will apply the patchset tomorrow if no objection.

-- 
Thanks,
Limin Wang
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

[FFmpeg-devel] [PATCH 1/2] avformat/dvenc: return error code of dv_assemble_frame()

2020-06-12 Thread lance . lmwang
From: Limin Wang 

Signed-off-by: Limin Wang 
---
 libavformat/dvenc.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/libavformat/dvenc.c b/libavformat/dvenc.c
index c71e532..b04d604 100644
--- a/libavformat/dvenc.c
+++ b/libavformat/dvenc.c
@@ -406,9 +406,10 @@ static int dv_write_packet(struct AVFormatContext *s, 
AVPacket *pkt)
 
 fsize = dv_assemble_frame(s, s->priv_data, s->streams[pkt->stream_index],
   pkt->data, pkt->size, &frame);
-if (fsize > 0) {
-avio_write(s->pb, frame, fsize);
+if (fsize < 0) {
+return fsize;
 }
+avio_write(s->pb, frame, fsize);
 return 0;
 }
 
-- 
1.8.3.1

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

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".