diff options
author | Tom Lane <tgl@sss.pgh.pa.us> | 2008-07-23 17:29:53 +0000 |
---|---|---|
committer | Tom Lane <tgl@sss.pgh.pa.us> | 2008-07-23 17:29:53 +0000 |
commit | 11c794f2249ba6b1d890da309e26e0ddff8bc60c (patch) | |
tree | 40d0a6fdad1bb8fd5350d4034ca7feebb312f416 /src/backend/executor/execUtils.c | |
parent | 509303a597a1688e6c2230c83634a75f872d270c (diff) | |
download | postgresql-11c794f2249ba6b1d890da309e26e0ddff8bc60c.tar.gz postgresql-11c794f2249ba6b1d890da309e26e0ddff8bc60c.zip |
Use guc.c's parse_int() instead of pg_atoi() to parse fillfactor in
default_reloptions(). The previous coding was really a bug because pg_atoi()
will always throw elog on bad input data, whereas default_reloptions is not
supposed to complain about bad input unless its validate parameter is true.
Right now you could only expose the problem by hand-modifying
pg_class.reloptions into an invalid state, so it doesn't seem worth
back-patching; but we should get it right in HEAD because there might be other
situations in future. Noted while studying GIN fast-update patch.
Diffstat (limited to 'src/backend/executor/execUtils.c')
0 files changed, 0 insertions, 0 deletions