Conditional jump or move depends on uninitialised value(s)

[strk]

According to valgrind, test_lwgeom_distance_sphere shows uninitialized values errors:

==7337== Conditional jump or move depends on uninitialised value(s)
==7337==    at 0x4E65C04: gbox_merge (g_box.c:231)
==7337==    by 0x4E6BE8E: ptarray_calculate_gbox_geodetic (lwgeodetic.c:2521)
==7337==    by 0x4E6C009: lwgeom_calculate_gbox_geodetic (lwgeodetic.c:2553)
==7337==    by 0x4E6DCDE: lwgeom_distance_spheroid (lwgeodetic.c:2115)
==7337==    by 0x40DB32: test_lwgeom_distance_sphere (cu_geodetic.c:1234)
==7337==    by 0x509BBAA: ??? (in /usr/lib/libcunit.so.1.0.1)
==7337==    by 0x509BE85: ??? (in /usr/lib/libcunit.so.1.0.1)
==7337==    by 0x509C19F: CU_run_all_tests (in /usr/lib/libcunit.so.1.0.1)
==7337==    by 0x4050EF: main (cu_tester.c:136)
==7337== 

[strk]

2.1 branch (pre 2.1.4) is also affected

[strk]

To reproduce:

cd liblwgeom/cunit && libtool --mode=execute valgrind --workaround-gcc296-bugs=yes ./cu_tester test_lwgeom_distance_sphere

[strk]

2.0 branch (pre 2.0.7) is also affected

[pramsey]

init edge box first?

[pramsey]

Does this patch help? I don't have valgrind on OSX at the moment.

[strk]

It does,test_lwgeom_distance_sphere runs with no memory errors applying that patch (still has some leaks, but that's a different issue)

[strk]

I've committed the patch in 2.0 branch as r12673, 2.1 branch as r12674, trunk as r12676