Boszormenyi Zoltan <zb@xxxxxxxxxxx> writes: > // ids = > PointerGetDatum(PG_DETOAST_DATUM(SPI_getbinval(prod_inv->vals[0], > prod_inv->tupdesc, 1, &isnull))); well, for one thing, you probably want DatumGetPointer ... for another, you shouldn't really be converting to Pointer here at all, since the next line expects ids to still be a Datum. On some platforms you can get away with being fuzzy about the distinction between Datum and Pointer, but it doesn't surprise me in the least that such code would fail elsewhere. Try paying more attention to compiler warnings ;-) The lack of any checks for null-ness scares me, too. Aside from the multiple places where you're just plain ignoring an isnull return flag, there's the risk that the array might contain one or more nulls, in which case you can't address the last element that way (even if that element itself isn't null). regards, tom lane -- Sent via pgsql-general mailing list (pgsql-general@xxxxxxxxxxxxxx) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-general
