On Mon, Jun 22, 2020 at 11:08:23PM +0200, Marton Balint wrote: > > > On Mon, 22 Jun 2020, [email protected] wrote: > > > From: Limin Wang <[email protected]> > > > > Signed-off-by: Limin Wang <[email protected]> > > --- > > libavformat/ftp.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/libavformat/ftp.c b/libavformat/ftp.c > > index caeea42..39ea297 100644 > > --- a/libavformat/ftp.c > > +++ b/libavformat/ftp.c > > @@ -200,7 +200,7 @@ static int ftp_status(FTPContext *s, char **line, const > > int response_codes[]) > > } > > > > if (line) > > - av_bprint_finalize(&line_buffer, line); > > + result = av_bprint_finalize(&line_buffer, line); > > No. ftp_status is not a function which returns 0 on success, it returns the > FTP status code. Please be more careful before submitting patches.
Sorry, please ignore the patch. I'm not clear how to map the error so I'll not update the patch anyore. > > Regards, > Marton > _______________________________________________ > ffmpeg-devel mailing list > [email protected] > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > To unsubscribe, visit link above, or email > [email protected] with subject "unsubscribe". -- Thanks, Limin Wang _______________________________________________ ffmpeg-devel mailing list [email protected] https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email [email protected] with subject "unsubscribe".
