babelfish-for-postgresql / babelfish_extensions / 21294394251

77%

push

github

BCP fix done token spanning packet boundaries (#4478)

Problem: When DONE token (13 bytes) was split across TDS packet boundaries, only the first byte (0xFD) would be checked, causing the remaining bytes in the next packet to be misinterpreted.
Actually we have always seen that done token gets truncated to fit in available space, I can see in some cases the done token is only 8 bytes at the end of the message - packet marked with EOM. If we're at EOM with incomplete DONE token, that's actually acceptable.

Solution:

Check if DONE token is present
Only fetch additional bytes if NOT at EOM and buffer has < 13 bytes
Accept incomplete DONE tokens at EOM (handles truncated tokens from optimizing clients)

JIRA: BABEL-6317

4 of 5 new or added lines in 1 file covered. (80.0%)

52708 of 68485 relevant lines covered (76.96%)

386804.14 hits per line