From: | tgl(at)postgresql(dot)org (Tom Lane) |
---|---|
To: | pgsql-committers(at)postgresql(dot)org |
Subject: | pgsql: Remove typmod checking from the recent security-related patches. |
Date: | 2007-02-06 17:35:20 |
Message-ID: | 20070206173520.B0C0B9FA434@postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
Log Message:
-----------
Remove typmod checking from the recent security-related patches. It turns
out that ExecEvalVar and friends don't necessarily have access to a tuple
descriptor with correct typmod: it definitely can contain -1, and possibly
might contain other values that are different from the Var's value.
Arguably this should be cleaned up someday, but it's not a simple change,
and in any case typmod discrepancies don't pose a security hazard.
Per reports from numerous people :-(
I'm not entirely sure whether the failure can occur in 8.0 --- the simple
test cases reported so far don't trigger it there. But back-patch the
change all the way anyway.
Modified Files:
--------------
pgsql/src/backend/executor:
execQual.c (r1.212 -> r1.213)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/executor/execQual.c.diff?r1=1.212&r2=1.213)
execUtils.c (r1.143 -> r1.144)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/executor/execUtils.c.diff?r1=1.143&r2=1.144)
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2007-02-06 17:35:27 | pgsql: Remove typmod checking from the recent security-related patches. |
Previous Message | Bruce Momjian | 2007-02-06 16:44:14 | pgsql: Split apart entries, one done now: * -Move NAMEDATALEN from |