[FFmpeg-devel] [PATCH] lavfi/vf_overlay: check that filter_frame does not return EAGAIN.

Michael Niedermayer michaelni at gmx.at
Thu Jan 10 20:57:29 CET 2013


On Thu, Jan 10, 2013 at 08:11:41PM +0100, Nicolas George wrote:
> EAGAIN is used internally in the code; it currently does not make
> sens as a return code from the next filter.
> 
> Should fix CID966655.
> 
> Signed-off-by: Nicolas George <nicolas.george at normalesup.org>
> ---
>  libavfilter/vf_overlay.c |    1 +
>  1 file changed, 1 insertion(+)
> 
> 
> I would prefer to test whether it convinces Coverity without pushing it
> to the official repository, but I do not know whether it is possible.

not worth it, overall work exceeds usefullness
if it doesnt fix it we just mark it as false positive
any attempts to test this without a push will take more time

and patch LGTM

 [...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

If you think the mosad wants you dead since a long time then you are either
wrong or dead since a long time.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130110/8fbeb433/attachment.asc>


More information about the ffmpeg-devel mailing list