[FFmpeg-devel] [PATCH] dec/developer: Add Valgrind / Address Sanitizer to the patch checklist

Paul B Mahol onemda at gmail.com
Sat Jan 12 16:37:18 CET 2013


On 1/12/13, Michael Niedermayer <michaelni at gmx.at> wrote:
> Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
> ---
>  doc/developer.texi |    3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/doc/developer.texi b/doc/developer.texi
> index 7e4b12c..0439a50 100644
> --- a/doc/developer.texi
> +++ b/doc/developer.texi
> @@ -507,6 +507,9 @@ send a reminder by email. Your patch should eventually
> be dealt with.
>      Make sure you check the return values of function and return
> appropriate
>      error codes. Especially memory allocation functions like
> @code{av_malloc()}
>      are notoriously left unchecked, which is a serious problem.
> + at item
> +    Test your code with valgrind and or Address Sanitizer to ensure its
> free
> +    of leaks, out of array accesses, etc.
>  @end enumerate
>
>  @section Patch review process
> --
> 1.7.9.5

fine for me


More information about the ffmpeg-devel mailing list