Skip to content

[Bug #111630]Add support for BigEndian clients in MySQL.Data #54

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

Closed
wants to merge 1 commit into from

Conversation

sid8606
Copy link

@sid8606 sid8606 commented Jun 30, 2023

BitConverter methods used in classes defined under the MySqlClient namespace inherently check for the endianness of the client system which leads to incorrect reads of incoming MySql packets sent by the server for BigEndian client machines.

Implemented a PacketBitConverter class to read incoming packets in LittleEndian order always.

BitConverter.ToString is unchanged since string values are read byte by byte.

BitConverter used in packages like NetworkStream and CompressedStream should not be replaced as it interfaces with network libraries.

@mysql-oca-bot
Copy link

Hi, thank you for submitting this pull request. In order to consider your code we need you to sign the Oracle Contribution Agreement (OCA). Please review the details and follow the instructions at https://oca.opensource.oracle.com/
Please make sure to include your MySQL bug system user (email) in the returned form.
Thanks

@sid8606
Copy link
Author

sid8606 commented Jul 7, 2023

/covered

@giritrivedi
Copy link

Can anyone start the review please ?

@giritrivedi
Copy link

@davaldezr . Can some review this ?

@mysql-oca-bot
Copy link

Hi, there was no response to our request to sign an OCA or confirm the code is submitted under the terms of the OCA. As such this request will be closed.
Thanks

@saitama951
Copy link

saitama951 commented Aug 18, 2023

@mysql-admin The OCA is signed, can you please confirm it and reopen the PR?

@mysql-admin
Copy link

Reopening to continue processing

Thanks for contributing to MySQL
==Omer

@mysql-admin mysql-admin reopened this Aug 18, 2023
@mysql-oca-bot
Copy link

Hi, thank you for your contribution. Please confirm this code is submitted under the terms of the OCA (Oracle's Contribution Agreement) you have previously signed by cutting and pasting the following text as a comment:
"I confirm the code being submitted is offered under the terms of the OCA, and that I am authorized to contribute it."
Thanks

@sid8606
Copy link
Author

sid8606 commented Aug 18, 2023

/covered

@sid8606
Copy link
Author

sid8606 commented Aug 18, 2023

I confirm the code being submitted is offered under the terms of the OCA, and that I am authorized to contribute it.

BitConverter methods used in classes defined under the MySqlClient
namespace inherently check for the endianness of the client system
which leads to incorrect reads of incoming MySql packets sent by the
server for BigEndian client machines.

Implemented a PacketBitConverter class to read incoming packets
in LittleEndian order always.

BitConverter.ToString is unchanged since string values are read byte
by byte.

BitConverter used in packages like NetworkStream and CompressedStream
should not be replaced as it interfaces with network libraries.

Signed-off-by: Sidraya <sidraya.jayagond@ibm.com>
@mysql-oca-bot
Copy link

Hi, thank you for your contribution. Your code has been assigned to an internal queue. Please follow
bug http://bugs.mysql.com/bug.php?id=111630 for updates.
Thanks

@saitama951
Copy link

@mysql-admin Hi, can we get the status on the PR by the verification team?

@mysql-admin
Copy link

Hi @saitama951 ,
This contribution is no tracked in http://bugs.mysql.com/bug.php?id=111630 you should ask for status via the bug system
Thanks
==Omer

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.

5 participants