Skip to content

SR-606: test_deserialize_allowFragments transacts upon uninitialized memory #2557

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

Merged
merged 1 commit into from
Nov 13, 2019

Conversation

spevans
Copy link
Contributor

@spevans spevans commented Nov 6, 2019

  • Re-enable test_deserialize_allowFragments_withData() and
    test_deserialize_allowFragments_withStream()

  • Fix deserialize_allowFragments() to pass .allowFragments allowing
    the tests to work correctly. Also test failure when .allowFragments
    is not passed.

  • Validate the tests against Darwin Foundation.

  • SR-446: valgrind now shows no invalid read in JSONReader.parseNumber
    in test TestJSONSerialization.test_deserialize_allowFragments()

…memory

- Re-enable test_deserialize_allowFragments_withData() and
  test_deserialize_allowFragments_withStream()

- Fix deserialize_allowFragments() to pass .allowFragments allowing
  the tests to work correctly. Also test failure when .allowFragments
  is not passed.

- Validate the tests against Darwin Foundation.

- SR-446: valgrind now shows no invalid read in JSONReader.parseNumber
  in test TestJSONSerialization.test_deserialize_allowFragments()
@spevans
Copy link
Contributor Author

spevans commented Nov 6, 2019

@swift-ci test

@spevans
Copy link
Contributor Author

spevans commented Nov 6, 2019

@swift-ci test macos

6 similar comments
@spevans
Copy link
Contributor Author

spevans commented Nov 7, 2019

@swift-ci test macos

@spevans
Copy link
Contributor Author

spevans commented Nov 7, 2019

@swift-ci test macos

@spevans
Copy link
Contributor Author

spevans commented Nov 8, 2019

@swift-ci test macos

@spevans
Copy link
Contributor Author

spevans commented Nov 9, 2019

@swift-ci test macos

@spevans
Copy link
Contributor Author

spevans commented Nov 11, 2019

@swift-ci test macos

@spevans
Copy link
Contributor Author

spevans commented Nov 13, 2019

@swift-ci test macos

@spevans
Copy link
Contributor Author

spevans commented Nov 13, 2019

@swift-ci test linux

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.

1 participant