Обсуждение: fmgr_oldstyle calls from 8.0.0RC2 trigger

Поиск
Список
Период
Сортировка

fmgr_oldstyle calls from 8.0.0RC2 trigger

От
strk
Дата:
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/


Re: fmgr_oldstyle calls from 8.0.0RC2 trigger

От
strk
Дата:
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/