eelke/pgLab
1
0
Fork 0
Code Issues 28 Pull requests Projects Releases 1 Packages Wiki Activity Actions

Don't read relhasoids on version 12. Also don't try to select data we are not allowed to read.

Browse source
This commit is contained in:
eelke 2019-08-10 18:12:26 +02:00
parent 1c48b1945c
commit 5494e5076b
3 changed files with 51 additions and 25 deletions
Download patch file Download diff file Expand all files Collapse all files

22
pglablib/catalog/PgClassContainer.cpp
View file

@ -7,13 +7,20 @@
std::string PgClassContainer::getLoadQuery() const
{
return "SELECT oid, relname, relnamespace, reltype, reloftype, "
std::string q = "SELECT oid, relname, relnamespace, reltype, reloftype, "
" relowner, relam, relfilenode, reltablespace, relpages, "
" reltuples, reltoastrelid, relisshared, relpersistence, "
" relkind, relhasoids, relispopulated, relfrozenxid, relminmxid, "
" reloptions, relacl \n"
"FROM pg_catalog.pg_class \n"
" relkind, relispopulated, relfrozenxid, relminmxid, "
" reloptions, relacl";
if (lessThenVersion(120000))
q += ", relhasoids ";
q +=
"\nFROM pg_catalog.pg_class \n"
"WHERE relkind IN ('r', 'i', 'p', 'I', 'v', 'm', 'f')";
return q;
}
PgClass PgClassContainer::loadElem(const Pgsql::Row &row)
@ -28,13 +35,18 @@ PgClass PgClassContainer::loadElem(const Pgsql::Row &row)
col >> v.type >> v.oftype
>> owner >> v.am >> v.filenode >> v.tablespace >> v.pages_est
>> v.tuples_est >> v.toastrelid >> v.isshared >> v.persistence
>> v.kind >> v.hasoids >> v.ispopulated >> v.frozenxid >> v.minmxid
>> v.kind >> v.ispopulated >> v.frozenxid >> v.minmxid
>> v.options;
v.setOwnerOid(owner);
AclList acl_list;
col >> acl_list;
v.setAcls(std::move(acl_list));
if (lessThenVersion(120000))
col >> v.hasoids;
return v;
}

4
pglablib/catalog/PgSequence.h
View file

@ -16,6 +16,10 @@ public:
bool cycled = false;
bool called = false;
bool priv_usage = false;
bool priv_select = false;
bool priv_update = false;
using PgClass::PgClass;
QString createSql() const;
};

18
pglablib/catalog/PgSequenceContainer.cpp
View file

@ -10,11 +10,17 @@ std::string PgSequenceContainer::getLoadQuery() const
std::string select = "SELECT pg_class.oid, relname, relnamespace, reltype, reloftype, "
" relowner, relam, relfilenode, reltablespace, relpages, "
" reltuples, reltoastrelid, relisshared, relpersistence, "
" relkind, relhasoids, relispopulated, relfrozenxid, relminmxid, "
" reloptions, relacl \n";
" relkind, relispopulated, relfrozenxid, relminmxid, "
" reloptions, relacl, \n"
" has_sequence_privilege(pg_class.oid, 'USAGE') AS priv_usage, \n"
" has_sequence_privilege(pg_class.oid, 'SELECT') AS priv_select, \n"
" has_sequence_privilege(pg_class.oid, 'UPDATE') AS priv_update \n";
std::string from =
"FROM pg_catalog.pg_class \n";
// Starting with 12 oid are not a special column anymore
if (lessThenVersion(120000))
select += ", relhasoids ";
// Starting with pg10 constanst data of sequences is retrieved from pg_sequence
// instead of SELECTing the sequence
if (minimumVersion(100000)) {
@ -43,8 +49,10 @@ PgSequence PgSequenceContainer::loadElem(const Pgsql::Row &row)
col >> v.type >> v.oftype
>> owner >> v.am >> v.filenode >> v.tablespace >> v.pages_est
>> v.tuples_est >> v.toastrelid >> v.isshared >> v.persistence
>> v.kind >> v.hasoids >> v.ispopulated >> v.frozenxid >> v.minmxid
>> v.options >> acl_list;
>> v.kind >> v.ispopulated >> v.frozenxid >> v.minmxid
>> v.options >> acl_list >> v.priv_usage >> v.priv_select >> v.priv_update;
if (lessThenVersion(120000))
col >> v.hasoids;
// Read pg_sequence fields
if (minimumVersion(100000))
col >> v.typid >> v.start >> v.increment >> v.max >> v.min >> v.cache >> v.cycled;
@ -60,6 +68,7 @@ void PgSequenceContainer::loadAll(Pgsql::Connection &conn)
IPgContainer::loadAll(conn);
for (auto && seq : m_container) {
if (seq.priv_select) {
auto fqn = seq.fullyQualifiedQuotedObjectName().toUtf8();
// SELECTs on a sequence return less fields starting with pg10
// the missing fields are now in pg_sequence
@ -81,3 +90,4 @@ void PgSequenceContainer::loadAll(Pgsql::Connection &conn)
}
}
}
}