Re: SPI_getvalue gives a core dump

This is what i set my tupdesc to. =>  tupdesc = trigdata->tg_relation->rd_att;
i am using the code like this. the do_log function to log the flow. it gives all correct until it reaches 
the line :-
disposition=SPI_getvalue(trigdata->tg_trigtuple, tupdesc, 13);               do_log("Got disposition");         ======> it stops logging from here.

               do_log("Entered the main trigger logic");  =========> success               if (TRIGGER_FIRED_BEFORE(trigdata->tg_event)) when = "before";               else when = "after ";
               tupdesc = trigdata->tg_relation->rd_att;               if ((ret = SPI_connect()) < 0)                       do_log("SPI_connect failed");               else {                       sprintf(logs,"SPI_connect success, fired %s",when);  ======> success                       do_log(logs);               }               ret = SPI_exec("SELECT count(*) FROM ast_cdr", 0);
               if (ret < 0){                       sprintf(logs, "trigf (fired %s): SPI_exec returned %d", when, ret);                       do_log(logs);               }
               /* count(*) returns int8, so be careful to convert */               i = DatumGetInt64(SPI_getbinval(SPI_tuptable->vals[0], tupdesc, 1, &isnull));               sprintf(logs, "trigf (fired %s): there are %d rows in ast_cdr", when, i); ====> success               do_log(logs);
                              disposition=SPI_getvalue(trigdata->tg_trigtuple, tupdesc, 13);  =====> failed.               do_log("Got disposition");                          ===============> doesnot reach here.




With warm regards.

Vivek J. Joshi.

vivek@staff.ownmail.com
Trikon electronics Pvt. Ltd.

All science is either physics or stamp collecting.               -- Ernest Rutherford



Tom Lane wrote:
> > writes:
>> I dont know where am i going wrong. Here is the source.
>
>> TriggerData *trigdata = (TriggerData *) fcinfo->context;
>> TupleDesc   tupdesc;
>> if (CALLED_AS_TRIGGER(fcinfo)) {  ==================== it comes here and 
enters inside.
>>     userfield=SPI_getvalue(trigdata->tg_trigtuple,tupdesc,17); 
================== it fails here because i tried to get logs before this 
statement and it succeeded. 
>
>Wrong tupdesc maybe?  You didn't show us what you are setting tupdesc
>to.
>
>			regards, tom lane
>
>---------------------------(end of broadcast)---------------------------
>TIP 4: Have you searched our list archives?
>
>               http://archives.postgresql.org