diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2014-12-10 19:06:27 -0500 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2014-12-10 19:06:27 -0500 |
| commit | 24688f4e5a7d5fadde0e43a5b123432d81577d82 (patch) | |
| tree | 8d42cf1bbf30e59ea8461464c9ecec625d02fd1c /src/backend/utils/adt/jsonb_util.c | |
| parent | e39250c644ea7cd3904e4e24570db21a209cf97f (diff) | |
| download | postgresql-24688f4e5a7d5fadde0e43a5b123432d81577d82.tar.gz | |
Fix minor thinko in convertToJsonb().
The amount of space to reserve for the value's varlena header is
VARHDRSZ, not sizeof(VARHDRSZ). The latter coding accidentally
failed to fail because of the way the VARHDRSZ macro is currently
defined; but if we ever change it to return size_t (as one might
reasonably expect it to do), convertToJsonb() would have failed.
Spotted by Mark Dilger.
Diffstat (limited to 'src/backend/utils/adt/jsonb_util.c')
| -rw-r--r-- | src/backend/utils/adt/jsonb_util.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/backend/utils/adt/jsonb_util.c b/src/backend/utils/adt/jsonb_util.c index 2ff85396d0..c62941baa7 100644 --- a/src/backend/utils/adt/jsonb_util.c +++ b/src/backend/utils/adt/jsonb_util.c @@ -1377,7 +1377,7 @@ convertToJsonb(JsonbValue *val) initStringInfo(&buffer); /* Make room for the varlena header */ - reserveFromBuffer(&buffer, sizeof(VARHDRSZ)); + reserveFromBuffer(&buffer, VARHDRSZ); convertJsonbValue(&buffer, &jentry, val, 0); |