Skip to content

[runtime] Fix 'skip_nested_brace' to not read past the string. Fixes #15253. - #15257

Merged
rolfbjarne merged 1 commit into
dotnet:mainfrom
rolfbjarne:issue-15253
Jun 14, 2022
Merged

rolfbjarne merged 1 commit into
dotnet:mainfrom
rolfbjarne:issue-15253

Conversation

@rolfbjarne

Copy link
Copy Markdown
Member

Fix 'skip_nested_brace' to not double skip characters.

Also add a test.

Fixes #15253.

…otnet#15253.

Fix 'skip_nested_brace' to not double skip characters.

Also add a test.

Fixes dotnet#15253.
@rolfbjarne rolfbjarne added the bug If an issue is a bug or a pull request a bug fix label Jun 13, 2022
@vs-mobiletools-engineering-service2

Copy link
Copy Markdown
Collaborator

💻 [PR Build] Tests on macOS Mac Catalina (10.15) passed 💻

✅ All tests on macOS Mac Catalina (10.15) passed.

Pipeline on Agent
Hash: c7de9a6cd7e00109f792efca6738ff25ec247ab0 [PR build]

@vs-mobiletools-engineering-service2

Copy link
Copy Markdown
Collaborator

📚 [PR Build] Artifacts 📚

Packages generated

View packages

Pipeline on Agent XAMBOT-1033.Monterey'
Hash: c7de9a6cd7e00109f792efca6738ff25ec247ab0 [PR build]

@vs-mobiletools-engineering-service2

Copy link
Copy Markdown
Collaborator

✅ API diff for current PR / commit

Legacy Xamarin (No breaking changes)
  • iOS (no change detected)
  • tvOS (no change detected)
  • watchOS (no change detected)
  • macOS (no change detected)
NET (empty diffs)
  • iOS: (empty diff detected)
  • tvOS: (empty diff detected)
  • MacCatalyst: (empty diff detected)
  • macOS: (empty diff detected)

✅ API diff vs stable

Legacy Xamarin (No breaking changes)
.NET (No breaking changes)
Legacy Xamarin (stable) vs .NET

✅ Generator diff

Generator diff is empty

Pipeline on Agent
Hash: c7de9a6cd7e00109f792efca6738ff25ec247ab0 [PR build]

@vs-mobiletools-engineering-service2

Copy link
Copy Markdown
Collaborator

❌ [PR Build] Tests on macOS M1 - Mac Big Sur (11.5) failed ❌

Failed tests are:

  • xammac_tests
  • monotouch-test

Pipeline on Agent
Hash: c7de9a6cd7e00109f792efca6738ff25ec247ab0 [PR build]

@vs-mobiletools-engineering-service2

Copy link
Copy Markdown
Collaborator

❌ [CI Build] Tests failed on VSTS: simulator tests iOS ❌

Tests failed on VSTS: simulator tests iOS.

Test results

1 tests failed, 145 tests passed.

Failed tests

  • fsharp/watchOS 32-bits - simulator/Debug: Crashed

Pipeline on Agent XAMBOT-1037.Monterey'
Merge c7de9a6 into 0fa87fe

@dalexsoto dalexsoto left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

whoah what a subtle one 👍

@rolfbjarne

Copy link
Copy Markdown
Member Author

Test failure is unrelated (https://github.com/xamarin/maccore/issues/2558).

@rolfbjarne
rolfbjarne merged commit 8adbefd into dotnet:main Jun 14, 2022
@rolfbjarne
rolfbjarne deleted the issue-15253 branch June 14, 2022 12:26
@rolfbjarne

Copy link
Copy Markdown
Member Author

/sudo backport release/6.0.3xx

@rolfbjarne

Copy link
Copy Markdown
Member Author

/sudo backport release/6.0.4xx

@vs-mobiletools-engineering-service2

Copy link
Copy Markdown
Collaborator

Backport Job to branch release/6.0.3xx Created! The magic is happening here

@vs-mobiletools-engineering-service2

Copy link
Copy Markdown
Collaborator

Backport Job to branch release/6.0.4xx Created! The magic is happening here

@vs-mobiletools-engineering-service2

Copy link
Copy Markdown
Collaborator

Hooray! Backport succeeded! Please see https://devdiv.visualstudio.com/DevDiv/_build/results?buildId=6400947 for more details.

@vs-mobiletools-engineering-service2

Copy link
Copy Markdown
Collaborator

Hooray! Backport succeeded! Please see https://devdiv.visualstudio.com/DevDiv/_build/results?buildId=6400946 for more details.

rolfbjarne added a commit that referenced this pull request Jul 14, 2022
…the string. Fixes #15253. (#15462)

Fix 'skip_nested_brace' to not double skip characters.

Also add a test.

Fixes #15253.


Backport of #15257

Co-authored-by: Rolf Bjarne Kvinge <rolf@xamarin.com>
rolfbjarne added a commit that referenced this pull request Jul 14, 2022
…the string. Fixes #15253. (#15461)

Fix 'skip_nested_brace' to not double skip characters.

Also add a test.

Fixes #15253.


Backport of #15257

Co-authored-by: Rolf Bjarne Kvinge <rolf@xamarin.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug If an issue is a bug or a pull request a bug fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Crash in native code with keyboard viewer + japan keyboard

6 participants