"Matthew T. O'Connor" <[EMAIL PROTECTED]> writes:
> pg_autovacuum.c has some problems with int overflow and not using appropriate
> datatypes to track certain table values.  This patch attempts to fix all 
> these issues.  Someone should take a look and make sure its valid.
> !     new_tbl->relid = atol(PQgetvalue(res, row, PQfnumber(res, "oid")));
> !     new_tbl->reltuples = atof(PQgetvalue(res, row, PQfnumber(res, "reltuples")));
> !     new_tbl->relpages = atol(PQgetvalue(res, row, PQfnumber(res, "relpages")));

This ignores the fact that relid and relpages are unsigned.  I would
suggest adopting the same convention for OID as is used in pg_dump and
other places:

#define atooid(x)  ((Oid) strtoul((x), NULL, 10))

You could actually use this same macro for reading relpages, but that's
probably abusing the notation.  I'd use strtoul directly for relpages,
I think.

> ! init_dbinfo(char *dbname, int oid, int age)
> ...
> ! init_dbinfo(char *dbname, uint oid, uint age)

This (and other declarations) should be "Oid oid".

                        regards, tom lane

---------------------------(end of broadcast)---------------------------
TIP 2: you can get off all lists at once with the unregister command
    (send "unregister YourEmailAddressHere" to [EMAIL PROTECTED])

Reply via email to