-
Notifications
You must be signed in to change notification settings - Fork 544
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
OpenXmlElement.CreateXmlReader: Prevent XmlConvertingReaderFactory from using a disposed TextReader #940
Conversation
…om using a disposed XmlReader
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you include a regression test for this!
Thanks for the fix!
/azp run |
@twsouthwick I added a test that shows the problem I encountered, and confirms the fix. |
Is there anything else I can help with to get this merged? |
/azp run |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
@sorensenmatias I've been pretty busy and so the delay is on me here. Once the tests pass, this can be merged in. Thanks for the contribution! |
/azp run |
Thanks @twsouthwick. I am not sure I completely follow though, is there a problem with the tests currently? |
Yeah - we have warnings as error on, and there's a style warning that's getting flagged |
/azp run |
I've pushed a fix for it (needed to add a copyright) |
Thanks. I fixed one more warning regarding UTF8 BOM. |
/azp run |
OpenXmlElement.CreateXmlReader: Prevent XmlConvertingReaderFactory from using a disposed TextReader.