Skip to content

Conversation

@labkey-nicka
Copy link
Contributor

@labkey-nicka labkey-nicka commented Aug 14, 2025

Rationale

Add regression test coverage for Issue 53699.

Related Pull Requests

Changes

  • Regression test for BufferUnderflowException and length validation using multibyte characters.

Copy link
Contributor

@labkey-jeckels labkey-jeckels left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@labkey-nicka please take a quick peek at my commit that adds the prefix to ensure it's hitting different byte offsets.

@labkey-nicka
Copy link
Contributor Author

@labkey-nicka please take a quick peek at my commit that adds the prefix to ensure it's hitting different byte offsets.

I've verified the BufferUnderflowException occurs when running the test without the associated platform changes. Subsequently, verified test case passes with changes as well as in CI.

image

@labkey-nicka labkey-nicka merged commit 3cee692 into release25.7-SNAPSHOT Aug 15, 2025
7 checks passed
@labkey-nicka labkey-nicka deleted the 25.7_fb_underflow_53699 branch August 15, 2025 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants