diff options
| author | Diego Biurrun <diego@biurrun.de> | 2012-06-30 15:35:57 +0200 |
|---|---|---|
| committer | Diego Biurrun <diego@biurrun.de> | 2012-07-02 10:56:54 +0200 |
| commit | 4aad316bff610c34b6eb047644e616982bdde621 (patch) | |
| tree | 0ee0d8a0c007ed1dd4aaa55dfd2eb656296330e6 /doc/developer.texi | |
| parent | edf35d83429b1cba76ae8fd4717bdd5e6a9fe0eb (diff) | |
doc: Remind devs to check return values, especially for malloc() et al
Diffstat (limited to 'doc/developer.texi')
| -rw-r--r-- | doc/developer.texi | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/doc/developer.texi b/doc/developer.texi index 2e25f5fb0d..d36f53f191 100644 --- a/doc/developer.texi +++ b/doc/developer.texi @@ -488,6 +488,10 @@ send a reminder by email. Your patch should eventually be dealt with. @item Lines with similar content should be aligned vertically when doing so improves readability. +@item + Make sure you check the return values of function and return appropriate + error codes. Especially memory allocation functions like @code{malloc()} + are notoriously left unchecked, which is a serious problem. @end enumerate @section Patch review process |