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

Michael Niedermayer git at videolan.org
Sat Jan 12 19:26:16 CET 2013


ffmpeg | branch: master | Michael Niedermayer <michaelni at gmx.at> | Sat Jan 12 16:27:55 2013 +0100| [1c14c3412ea1bac5d02be284103c153599044564] | committer: Michael Niedermayer

dec/developer: Add Valgrind / Address Sanitizer to the patch checklist

Reviewed-by: saste
Reviewed-by: Paul B Mahol <onemda at gmail.com>
Signed-off-by: Michael Niedermayer <michaelni at gmx.at>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=1c14c3412ea1bac5d02be284103c153599044564
---

 doc/developer.texi |    3 +++
 1 file changed, 3 insertions(+)

diff --git a/doc/developer.texi b/doc/developer.texi
index 7e4b12c..a108da7 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 it's free
+    of leaks, out of array accesses, etc.
 @end enumerate
 
 @section Patch review process



More information about the ffmpeg-cvslog mailing list