Fix several issues when building cunit tests

- loader cunit: Add missing $(CFLAGS) to the main binary
- liblwgeom cunit: Avoid link issues with some sanitizers

References #4383

git-svn-id: http://svn.osgeo.org/postgis/trunk@17411 b70326c6-7e19-0410-871a-916f4a2858ee

diff --git a/liblwgeom/cunit/cu_in_twkb.c b/liblwgeom/cunit/cu_in_twkb.c
index 9313f13ad3d8ae08c654086597ade0848aa0aa35..43542b2178ae0174f7428234cac582cfcb27974e 100644 (file)
--- a/liblwgeom/cunit/cu_in_twkb.c
+++ b/liblwgeom/cunit/cu_in_twkb.c
@@ -20,10 +20,10 @@
 /*
 ** Global variable to hold TWKB strings
 */
-char *hex_a;
-char *hex_b;
-uint8_t precision = 0;
-uint8_t variant = 0;
+static char *hex_a;
+static char *hex_b;
+static uint8_t precision = 0;
+static uint8_t variant = 0;
 
 /*
 ** The suite initialization function.

diff --git a/liblwgeom/cunit/cu_in_wkb.c b/liblwgeom/cunit/cu_in_wkb.c
index 8e051f7e6f6ea288a4f9d241eb0d4337b158f2b1..bcd5e2891a4ab418506391e77f5d69ec6ffc3826 100644 (file)
--- a/liblwgeom/cunit/cu_in_wkb.c
+++ b/liblwgeom/cunit/cu_in_wkb.c
@@ -20,8 +20,8 @@
 /*
 ** Global variable to hold WKB strings
 */
-char *hex_a;
-char *hex_b;
+static char *hex_a;
+static char *hex_b;
 
 /*
 ** The suite initialization function.

diff --git a/liblwgeom/cunit/cu_out_twkb.c b/liblwgeom/cunit/cu_out_twkb.c
index 27f51e9d139d0ea11e12134251949dea65417ee1..3609416dbcd2df84d26b785e28054ce2ea515c01 100644 (file)
--- a/liblwgeom/cunit/cu_out_twkb.c
+++ b/liblwgeom/cunit/cu_out_twkb.c
@@ -21,8 +21,8 @@
 /*
 ** Global variable to hold hex TWKB strings
 */
-char *s;
-char *w;
+static char *s;
+static char *w;
 
 /*
 ** The suite initialization function.

diff --git a/liblwgeom/cunit/cu_out_wkb.c b/liblwgeom/cunit/cu_out_wkb.c
index d60db1b6509f766b079db753ca26fce813d55cb4..7b23fe634b14744e8d855ebe4b2b58d4981d4778 100644 (file)
--- a/liblwgeom/cunit/cu_out_wkb.c
+++ b/liblwgeom/cunit/cu_out_wkb.c
@@ -20,8 +20,8 @@
 /*
 ** Global variable to hold hex WKB strings
 */
-char *s;
-size_t s_size;
+static char *s;
+static size_t s_size;
 
 /*
 ** The suite initialization function.

diff --git a/liblwgeom/cunit/cu_out_wkt.c b/liblwgeom/cunit/cu_out_wkt.c
index c070efbd5e1f6dc7ee81310cbd1481a30ed7cf8f..199ca62b0c0613ea6168b65d9e0e16b72c5a073f 100644 (file)
--- a/liblwgeom/cunit/cu_out_wkt.c
+++ b/liblwgeom/cunit/cu_out_wkt.c
@@ -20,7 +20,7 @@
 /*
 ** Global variable to hold WKT strings
 */
-char *s = NULL;
+static char *s = NULL;
 
 /*
 ** The suite initialization function.

diff --git a/liblwgeom/cunit/cu_tester.c b/liblwgeom/cunit/cu_tester.c
index 88ab739819a352cc0b2d99c9dbd21c6c0fe4e929..3637226d56940544707dd515bce79df2835b42e4 100644 (file)
--- a/liblwgeom/cunit/cu_tester.c
+++ b/liblwgeom/cunit/cu_tester.c
@@ -16,6 +16,8 @@
 #include "cu_tester.h"
 #include "../postgis_config.h"
 
+char cu_error_msg[MAX_CUNIT_ERROR_LENGTH + 1] = {0};
+
 /* Internal funcs */
 static void
 cu_errorreporter(const char *fmt, va_list ap);

diff --git a/liblwgeom/cunit/cu_tester.h b/liblwgeom/cunit/cu_tester.h
index c5e6ccfac1e20904cb48c970fe74e4ea632188ae..862bc09e3dc4fb37b94bc3d275714d44c7f8b194 100644 (file)
--- a/liblwgeom/cunit/cu_tester.h
+++ b/liblwgeom/cunit/cu_tester.h
@@ -10,6 +10,9 @@
  *
  **********************************************************************/
 
+#ifndef _CU_TESTER_H
+#define _CU_TESTER_H 1
+
 #include "liblwgeom.h"
 
 #define MAX_CUNIT_ERROR_LENGTH 512
@@ -17,7 +20,7 @@
 #define PG_ADD_TEST(suite, testfunc) CU_add_test(suite, #testfunc, testfunc)
 
 /* Contains the most recent error message generated by lwerror. */
-char cu_error_msg[MAX_CUNIT_ERROR_LENGTH+1];
+extern char cu_error_msg[];
 
 /* Resets cu_error_msg back to blank. */
 void cu_error_msg_reset(void);
@@ -87,3 +90,5 @@ typedef void (*PG_SuiteSetup)(void);
 
 /* Utility functions */
 void do_fn_test(LWGEOM* (*transfn)(LWGEOM*), char *input_wkt, char *expected_wkt);
+
+#endif /* _CU_TESTER_H */

diff --git a/loader/cunit/Makefile.in b/loader/cunit/Makefile.in
index 253a1c05c3e8c8bd573418666ba749e3c213a64c..345bfe8e7701f9105a0bfafc6065eca0fad6bf76 100644 (file)
--- a/loader/cunit/Makefile.in
+++ b/loader/cunit/Makefile.in
@@ -84,7 +84,7 @@ endif
 # Build the main unit test executable
 cu_tester: $(LOADER_OBJS) $(OBJS)
        $(LIBTOOL) --mode=link \
-       $(CC)  $^ -o $@ $(LIBLWGEOM) $(LDFLAGS)
+       $(CC) $(CFLAGS) $^ -o $@ $(LIBLWGEOM) $(LDFLAGS)
 
 # Command to build each of the .o files
 $(OBJS): %.o: %.c