fmgr_oldstyle calls from 8.0.0RC2 trigger

Started by strkabout 21 years ago2 messages
#1strk
strk@keybit.net

I'm trying to implement a trigger within 8.0.0RC2.
I get a segfault when the trigger is fired, valgrind
reports an illegal read being the line:

TriggerData *trigdata = (TriggerData *) fcinfo->context;

Valgrind reports:

==941== Use of uninitialised value of size 4
==941== at 0x3CD9C197: cache_bbox (lwgeom_triggers.c:10)
==941== by 0x825C274: fmgr_oldstyle (fmgr.c:637)
==941== by 0x81258C8: ExecCallTriggerFunc (trigger.c:1149)
==941== by 0x812646F: ExecBRUpdateTriggers (trigger.c:1525)

Isn't fmgr_oldstyle the Version 0 function caller ?
Shouldn't triggers be defined as Version 1 ?

TIA

--strk;

For standing up against patentability of software,

Thank You, Poland!

Read the intervention: http://kwiki.ffii.org/ConsPolon041221En
Send your thanks: thankyoupoland.info
Read/do more: http://www.noepatents.org/

#2strk
strk@keybit.net
In reply to: strk (#1)
Re: fmgr_oldstyle calls from 8.0.0RC2 trigger

Sorry, my fault, just mistyped the function name
in PG_FUNCTION_INFO_V1() macro.

--strk;

On Wed, Dec 29, 2004 at 11:08:28AM +0100, strk wrote:

I'm trying to implement a trigger within 8.0.0RC2.
I get a segfault when the trigger is fired, valgrind
reports an illegal read being the line:

TriggerData *trigdata = (TriggerData *) fcinfo->context;

Valgrind reports:

==941== Use of uninitialised value of size 4
==941== at 0x3CD9C197: cache_bbox (lwgeom_triggers.c:10)
==941== by 0x825C274: fmgr_oldstyle (fmgr.c:637)
==941== by 0x81258C8: ExecCallTriggerFunc (trigger.c:1149)
==941== by 0x812646F: ExecBRUpdateTriggers (trigger.c:1525)

Isn't fmgr_oldstyle the Version 0 function caller ?
Shouldn't triggers be defined as Version 1 ?

TIA

--strk;

For standing up against patentability of software,

Thank You, Poland!

Read the intervention: http://kwiki.ffii.org/ConsPolon041221En
Send your thanks: thankyoupoland.info
Read/do more: http://www.noepatents.org/

---------------------------(end of broadcast)---------------------------
TIP 7: don't forget to increase your free space map settings

--

For standing up against patentability of software,

Thank You, Poland!

Read the intervention: http://kwiki.ffii.org/ConsPolon041221En
Send your thanks: thankyoupoland.info
Read/do more: http://www.noepatents.org/