The cmocka.h header MUST be included before isc/util.h gets included

The isc/util.h header redefine the DbC checks (REQUIRE, INSIST, ...)  to
be cmocka "fake" assertions.  However that means that cmocka.h needs to
be included after UNIT_TESTING is defined but before isc/util.h is
included.  Because isc/util.h is included in most of the project headers
this means that the sequence MUST be:

    #define UNIT_TESTING
    #include <cmocka.h>

    #include <isc/_anything_.h>

See !2204 for other header requirements for including cmocka.h.

(cherry picked from commit 0ba697fe8c3a05fc7723529fa20ea9c7865e5b6c)

diff --git a/lib/isc/tests/task_test.c b/lib/isc/tests/task_test.c
index 9cbe1c99c69ebb0de6c439bff1e1eb2a83616af9..f4a44a12c57aeb6e3905925c98c093d8b83c60ad 100644 (file)
--- a/lib/isc/tests/task_test.c
+++ b/lib/isc/tests/task_test.c
@@ -23,6 +23,8 @@
 
 #define UNIT_TESTING
 
+#include <cmocka.h>
+
 #include <isc/atomic.h>
 #include <isc/cmocka.h>
 #include <isc/commandline.h>