diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2015-07-02 17:02:08 -0400 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2015-07-02 17:02:14 -0400 |
| commit | 69e9f9639d5c569a71c82f99550e7bf2912664f1 (patch) | |
| tree | d3d6f68e85358fdd28cb5919fa280940ee450a66 /src/bin/pg_archivecleanup/pg_archivecleanup.c | |
| parent | cf2b5f9b33fda1cbeb8efdfd3989b5e88af74167 (diff) | |
Fix misuse of TextDatumGetCString().
"TextDatumGetCString(PG_GETARG_TEXT_P(x))" is formally wrong: a text*
is not a Datum. Although this coding will accidentally fail to fail on
all known platforms, it risks leaking memory if a detoast step is needed,
unlike "TextDatumGetCString(PG_GETARG_DATUM(x))" which is what's used
elsewhere. Make pg_get_object_address() fall in line with other uses.
Noted while reviewing two-arg current_setting() patch.
Diffstat (limited to 'src/bin/pg_archivecleanup/pg_archivecleanup.c')
0 files changed, 0 insertions, 0 deletions