aboutsummaryrefslogtreecommitdiff
path: root/src/backend/replication/logical/reorderbuffer.c
diff options
context:
space:
mode:
authorTom Lane <tgl@sss.pgh.pa.us>2016-03-06 19:20:55 -0500
committerTom Lane <tgl@sss.pgh.pa.us>2016-03-06 19:20:55 -0500
commitb3e05097e58051a7816ed734074fd76345687e0c (patch)
tree653e827a89673e1afacb2fc92ab841e3b1b641c2 /src/backend/replication/logical/reorderbuffer.c
parentcb0ca0c9953aa0614e7b143bd2440a7582a27233 (diff)
downloadpostgresql-b3e05097e58051a7816ed734074fd76345687e0c.tar.gz
postgresql-b3e05097e58051a7816ed734074fd76345687e0c.zip
Fix not-terribly-safe coding in NIImportOOAffixes() and NIImportAffixes().
There were two places in spell.c that supposed that they could search for a location in a string produced by lowerstr() and then transpose the offset into the original string. But this fails completely if lowerstr() transforms any characters into characters of different byte length, as can happen in Turkish UTF8 for instance. We'd added some comments about this coding in commit 51e78ab4ff328296, but failed to realize that it was not merely confusing but wrong. Coverity complained about this code years ago, but in such an opaque fashion that nobody understood what it was on about. I'm not entirely sure that this issue *is* what it's on about, actually, but perhaps this patch will shut it up -- and in any case the problem is clear. Back-patch to all supported branches.
Diffstat (limited to 'src/backend/replication/logical/reorderbuffer.c')
0 files changed, 0 insertions, 0 deletions