Fix bogus code in contrib/ tsearch dictionary examples.
authorTom Lane <tgl@sss.pgh.pa.us>
Thu, 3 Nov 2011 23:18:10 +0000 (19:18 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Thu, 3 Nov 2011 23:18:10 +0000 (19:18 -0400)
commit0dddbbcd3178242f24d7476e12436ec7e1340f6d
tree5c8ea17c7421881fb3a17fa84f8ad85cf6cca509
parenta0bd4f7c2a69064f5fd3a519a7ad8897dbf0d63a
Fix bogus code in contrib/ tsearch dictionary examples.

Both dict_int and dict_xsyn were blithely assuming that whatever memory
palloc gives back will be pre-zeroed.  This would typically work for
just about long enough to run their regression tests, and no longer :-(.

The pre-9.0 code in dict_xsyn was even lamer than that, as it would
happily give back a pointer to the result of palloc(0), encouraging
its caller to access off the end of memory.  Again, this would just
barely fail to fail as long as memory contained nothing but zeroes.

Per a report from Rodrigo Hjort that code based on these examples
didn't work reliably.
contrib/dict_int/dict_int.c
contrib/dict_xsyn/dict_xsyn.c