CodingStyle: add some more error handling guidelines
authorDan Carpenter <dan.carpenter@oracle.com>
Tue, 2 Dec 2014 08:59:50 +0000 (11:59 +0300)
committerJonathan Corbet <corbet@lwn.net>
Tue, 2 Dec 2014 13:55:32 +0000 (08:55 -0500)
commitea04036032edda6f771c1381d03832d2ed0f6c31
tree3266fb1cff71f9e52440d2710b5f6b03a8f3b8d7
parent86d3e023e05d90b2b5f88dcbf2e334b5835131f8
CodingStyle: add some more error handling guidelines

I added a paragraph on choosing label names, and updated the example
code to use a better label name.  I also cleaned up the example code to
more modern style by moving the allocation out of the initializer and
changing the NULL check.

Perhaps the most common type of error handling bug in the kernel is "one
err bugs".  CodingStyle already says that we should "avoid nesting" by
using error labels and one err style error handling tends to have
multiple indent levels, so this was already bad style.  But I've added a
new paragraph explaining how to avoid one err bugs by using multiple
error labels which is, hopefully, more clear.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Julia Lawall <julia.lawall@lip6.fr>
[jc: added GFP_KERNEL to kmalloc() call]
Signed-off-by: Jonathan Corbet <corbet@lwn.net>
Documentation/CodingStyle