| From: | Heikki Linnakangas <heikki(dot)linnakangas(at)enterprisedb(dot)com> |
|---|---|
| To: | Boszormenyi Zoltan <zb(at)cybertec(dot)at> |
| Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-hackers(at)postgresql(dot)org, Hans-Juergen Schoenig <hs(at)cybertec(dot)at>, Leonardo Bispo de Oliveira <leonardo(dot)oliveira(at)voipfuture(dot)com> |
| Subject: | Re: DatumGetInetP buggy |
| Date: | 2011-11-08 20:52:43 |
| Message-ID: | 4EB9969B.7010609@enterprisedb.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On 08.11.2011 20:46, Boszormenyi Zoltan wrote:
> 2011-11-08 18:53 keltezéssel, Heikki Linnakangas írta:
>> What do people think of backpatching this? I'm inclined to backpatch, on the grounds
>> that the macro that detoasts and depacks should always work (ie. after this patch), even
>> if the depacking isn't necessary and introduces an extra palloc+copy, but the reverse is
>> not true.
>
> On the grounds that 9.0.x also shows the problem with
> the stock macro, backporting would be nice. I don't know
> which older trees may show the problem, I only tested with
> 9.0 and 9.1.
Packed varlenas were introduced in 8.3 - this hasn't changed since then.
Committed and backpatched all the way to 8.3.
--
Heikki Linnakangas
EnterpriseDB http://www.enterprisedb.com
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Josh Berkus | 2011-11-08 20:56:14 | Releasing an alpha for CF2 |
| Previous Message | Greg Jaskiewicz | 2011-11-08 19:46:44 | 9.1.2 ? |