summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLennart Poettering <lennart@poettering.net>2015-04-02 12:14:43 +0200
committerLennart Poettering <lennart@poettering.net>2015-04-02 12:17:41 +0200
commit3dbafa39b08025350e7b17f4874a343c789ff9b3 (patch)
tree3b0462703f72a8937b2d0dc0445f37ff587ccff1
parentf4ac4d1a82e2c468761fffa333323841ad886221 (diff)
CODING_STYLE: clarify that exit() is never OK to call
-rw-r--r--CODING_STYLE6
1 files changed, 6 insertions, 0 deletions
diff --git a/CODING_STYLE b/CODING_STYLE
index b687e729f4..1748dc4bc4 100644
--- a/CODING_STYLE
+++ b/CODING_STYLE
@@ -226,3 +226,9 @@
instead of just this:
unlink("/foo/bar/baz");
+
+- Don't invoke exit(), ever. It is not replacement for proper error
+ handling. Please escalate errors up your call chain, and use normal
+ "return" to exit from the main function of a process. If you
+ fork()ed off a child process, please use _exit() instead of exit(),
+ so that the exit handlers are not run.