You can not select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.

Tom Lane 53ae246923 Handle unexpected query results, especially NULLs, safely in connectby(). ... connectby() didn't adequately check that the constructed SQL query returns what it's expected to; in fact, since commit 08c33c426b it wasn't checking that at all. This could result in a null-pointer-dereference crash if the constructed query returns only one column instead of the expected two. Less excitingly, it could also result in surprising data conversion failures if the constructed query returned values that were not I/O-conversion-compatible with the types specified by the query calling connectby(). In all branches, insist that the query return at least two columns; this seems like a minimal sanity check that can't break any reasonable use-cases. In HEAD, insist that the constructed query return the types specified by the outer query, including checking for typmod incompatibility, which the code never did even before it got broken. This is to hide the fact that the implementation does a conversion to text and back; someday we might want to improve that. In back branches, leave that alone, since adding a type check in a minor release is more likely to break things than make people happy. Type inconsistencies will continue to work so long as the actual type and declared type are I/O representation compatible, and otherwise will fail the same way they used to. Also, in all branches, be on guard for NULL results from the constructed query, which formerly would cause null-pointer dereference crashes. We now print the row with the NULL but don't recurse down from it. In passing, get rid of the rather pointless idea that build_tuplestore_recursively() should return the same tuplestore that's passed to it. Michael Paquier, adjusted somewhat by me		11 years ago
..
data	Have crosstab variants treat NULL rowid as a category in its own right,	18 years ago
expected	Handle unexpected query results, especially NULLs, safely in connectby().	11 years ago
sql	Handle unexpected query results, especially NULLs, safely in connectby().	11 years ago
.gitignore	Support "make check" in contrib	14 years ago
Makefile	Convert contrib modules to use the extension facility.	15 years ago
tablefunc--1.0.sql	Throw a useful error message if an extension script file is fed to psql.	14 years ago
tablefunc--unpackaged--1.0.sql	Fix typos in some error messages thrown by extension scripts when fed to psql.	11 years ago
tablefunc.c	Handle unexpected query results, especially NULLs, safely in connectby().	11 years ago
tablefunc.control	Convert contrib modules to use the extension facility.	15 years ago
tablefunc.h	Update copyrights for 2013	13 years ago