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

[XrdCl] Avoid partially reading status message data too early #1419

Merged
merged 1 commit into from
Mar 8, 2021

Conversation

smithdh
Copy link
Contributor

@smithdh smithdh commented Mar 4, 2021

No description provided.

@abh3
Copy link
Member

abh3 commented Mar 4, 2021

I will leave it to Michal to approve though he won't be back in action until Monday. However, it you could provide what the erroneous result is that would be helpful (i.e. what kind of error results).

@smithdh
Copy link
Contributor Author

smithdh commented Mar 4, 2021

I've added an issue to describe the original problem, what was seen etc. Issue #1420. Thanks.

@abh3
Copy link
Member

abh3 commented Mar 4, 2021 via email

@simonmichal
Copy link
Contributor

@smithdh : good catch, thanks a lot!

@simonmichal simonmichal merged commit fa4051a into xrootd:master Mar 8, 2021
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.

None yet

3 participants