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.

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

GetSqlBinary fails with NRE on null rowversion #1684

Closed
Wraith2 opened this issue Jul 27, 2022 · 3 comments · Fixed by #1688
Closed

GetSqlBinary fails with NRE on null rowversion #1684

Wraith2 opened this issue Jul 27, 2022 · 3 comments · Fixed by #1688

Comments

@Wraith2
Copy link
Contributor

Wraith2 commented Jul 27, 2022

select cast(null as rowversion) rv

Wow that's impressive that you were able to boil it down to that. Very cool. Now just try to read that in with GetSqlBinary. It does in fact throw I just tried it.

image

Originally posted by @rclabo in #255 (comment)

@dbrownems
Copy link

GetSqlBytes fails too.

@DavoudEshtehari
Copy link
Contributor

Hi @Wraith2, it seems you hunt a new issue. 😃 We'll be happy to hear you're going to work on it.

FYI, we're supposed to release 3.1.1 hotfix that would be the version used by management studio team, and probably a proposed fix for this issue will include in that hotfix.

@Wraith2
Copy link
Contributor Author

Wraith2 commented Jul 27, 2022

I don't need to hunt for issues, they find me.
In this case it is likely that the original change I made needs to be extended to the SqlTypes. I'll try and have a look at this over the weekend.

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 a pull request may close this issue.

3 participants