Opened 6 years ago

Closed 6 years ago

#4095 closed defect (fixed)

Median fails robustness test on median

Reported by: robe Owned by: Algunenano
Priority: blocker Milestone: PostGIS 2.3.8
Component: postgis Version: 2.3.x
Keywords: Cc:

Description

After this r16579

I'm seeing on winnie's 32-bit 9.6 run so could be a 32-bit only issue. I'll try to rerun to see if the 64-bits exhibit similar issue

 Test: test_median_handles_3d_correctly ...passed
  Test: test_median_robustness ...make[2]: *** [Makefile:86: check] Segmentation fault
make[2]: Leaving directory '/projects/postgis/branches/2.4/liblwgeom/cunit'
make[1]: *** [Makefile:207: check] Error 2
make[1]: Leaving directory '/projects/postgis/branches/2.4/liblwgeom'
make: *** [GNUmakefile:16: check] Error 1

Change History (4)

comment:1 by robe, 6 years ago

Milestone: PostGIS 2.4.5PostGIS 2.3.8
Version: 2.4.x2.3.x

Same issue with 2.3 branch - r16580

comment:2 by Algunenano, 6 years ago

Owner: changed from pramsey to Algunenano

I've been able to reproduce it in Ubuntu 17.04 (32bits):

Program received signal SIGSEGV, Segmentation fault.
0xb7f5ba74 in lwpoint_is_empty (point=0x0) at lwpoint.c:293
293		if ( ! point->point || point->point->npoints < 1 )
(gdb) bt
#0  0xb7f5ba74 in lwpoint_is_empty (point=0x0) at lwpoint.c:293
#1  0xb7f5bb0a in lwpoint_getPoint3dz_p (point=0x0, out=0xbfffeeb8) at lwpoint.c:49
#2  0x0804d320 in do_median_test (input=<optimized out>, expected=<optimized out>) at cu_algorithm.c:1015
#3  0xb7ee1507 in ?? () from /usr/lib/i386-linux-gnu/libcunit.so.1
#4  0xb7ee1f76 in CU_run_test () from /usr/lib/i386-linux-gnu/libcunit.so.1
#5  0x0804cd55 in main (argc=<optimized out>, argv=<optimized out>) at cu_tester.c:216

I'll have a look tomorrow.

Last edited 6 years ago by Algunenano (previous) (diff)

comment:3 by Raul Marin, 6 years ago

In 16587:

Avoid segfault in median test (32bits)

References #4095

comment:4 by Raul Marin, 6 years ago

Resolution: fixed
Status: newclosed

Should be ok now.

Note: See TracTickets for help on using tickets.