pgsql: Make bit/varbit substring() treat any negative length as meaning

From: tgl(at)postgresql(dot)org (Tom Lane)
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Make bit/varbit substring() treat any negative length as meaning
Date: 2010-01-07 19:53:11
Message-ID: 20100107195311.7C4017541B9@cvs.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Log Message:
-----------
Make bit/varbit substring() treat any negative length as meaning "all the rest
of the string". The previous coding treated only -1 that way, and would
produce an invalid result value for other negative values.

We ought to fix it so that 2-parameter bit substring() is a different C
function and the 3-parameter form throws error for negative length, but
that takes a pg_proc change which is impractical in the back branches;
and in any case somebody might be relying on -1 working this way.
So just do this as a back-patchable fix.

Modified Files:
--------------
pgsql/src/backend/utils/adt:
varbit.c (r1.61 -> r1.62)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/adt/varbit.c?r1=1.61&r2=1.62)

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2010-01-07 19:53:16 pgsql: Make bit/varbit substring() treat any negative length as meaning
Previous Message Tom Lane 2010-01-07 16:29:58 pgsql: Fix (some of the) breakage introduced into query-cancel