Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Execution fails badly if there are more than 32767 parameters #2703

Closed
roji opened this issue Oct 21, 2019 · 0 comments · Fixed by #2705
Closed

Execution fails badly if there are more than 32767 parameters #2703

roji opened this issue Oct 21, 2019 · 0 comments · Fixed by #2705
Assignees
@roji
Labels
bug
Milestone
4.0.11

Comments

@roji
Copy link
Member

roji commented Oct 21, 2019 •
edited

We are erroneously checking if there are more than 65535 parameters (unsigned short), whereas the actual PostgreSQL upper value is 32767 (signed short).
@roji roji added the bug label Oct 21, 2019
@roji roji added this to the 4.0.11 milestone Oct 21, 2019
@roji roji self-assigned this Oct 21, 2019
@roji roji changed the title Prepare fails badly if more than 32767 parameters Execution fails badly if there are more than 32767 parameters Oct 21, 2019
roji added a commit that referenced this issue Oct 21, 2019
@roji
Correct parameter limit check
aff4acd 
Fixes #2703
@roji roji mentioned this issue Oct 21, 2019
Merged
roji added a commit that referenced this issue Oct 22, 2019
@roji
Correct parameter limit check
7a830c5 
Fixes #2703
roji added a commit that referenced this issue Oct 22, 2019
@roji
Correct parameter limit check
bf29cb7 
Fixes #2703

(cherry picked from commit 7a830c5)
roji added a commit that referenced this issue Oct 22, 2019
@roji
Correct parameter limit check
0970e0f 
Fixes #2703

(cherry picked from commit 7a830c5)
@roji roji mentioned this issue Jan 17, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@roji roji

Labels
bug
Projects
None yet
Milestone
4.0.11
Development

Successfully merging a pull request may close this issue.

Correct parameter limit check npgsql/npgsql
1 participant
@roji