Update pulldown-cmark to v0.13.0 to fix crash#46267
Update pulldown-cmark to v0.13.0 to fix crash#46267MrSubidubi merged 3 commits intozed-industries:mainfrom
Conversation
|
We require contributors to sign our Contributor License Agreement, and we don't have @drbh on file. You can sign our CLA at https://zed.dev/cla. Once you've signed, post a comment here that says '@cla-bot check'. |
|
@cla-bot check |
|
We require contributors to sign our Contributor License Agreement, and we don't have @drbh on file. You can sign our CLA at https://zed.dev/cla. Once you've signed, post a comment here that says '@cla-bot check'. |
|
The cla-bot has been summoned, and re-checked this pull request! |
|
@cla-bot check |
|
The cla-bot has been summoned, and re-checked this pull request! |
|
whoops apologies for the pr renaming I didn't see that you updated it @MrSubidubi! Thanks for the quick look |
MrSubidubi
left a comment
There was a problem hiding this comment.
Nice, thank you very much for looking into this and the detailed context! Much appreciated.
This comment was marked as outdated.
This comment was marked as outdated.
1 similar comment
|
/cherry-pick preview |
Hi Zed team thank you for the awesome editor!
I recently stumbled upon a markdown sequence that seems to cause a crash
```md
- [ ] -
\
-
```
*easier to read escape characters below
```rust
let crash_input = "-\t[\t] -\r\\\n-"
println!("{}", crash_input)
```
## how to reproduce
1. copy the markdown above
2. save the file
3. `[shift]` + `[cmd]` + `p` to open the command palette
4. select `markdown: open preview`
5. crash
I've confirmed that the issue is a bug in pulldown-cmark version 12, and
has been resolved in version
[v0.13.0](https://github.com/pulldown-cmark/pulldown-cmark/releases/tag/v0.13.0)
and specifically fixed in
pulldown-cmark/pulldown-cmark#1017
this PR simply bumps the pulldown-cmark version in zed which resolves
the crash on my local machine.
## recording
https://github.com/user-attachments/assets/dc77132f-0d43-40f3-9841-0bf34fe714fb
Release Notes:
- Fixes crash due to markdown parsing via bumping pulldown-cmark
|
/cherry-pick stable |
Hi Zed team thank you for the awesome editor!
I recently stumbled upon a markdown sequence that seems to cause a crash
```md
- [ ] -
\
-
```
*easier to read escape characters below
```rust
let crash_input = "-\t[\t] -\r\\\n-"
println!("{}", crash_input)
```
## how to reproduce
1. copy the markdown above
2. save the file
3. `[shift]` + `[cmd]` + `p` to open the command palette
4. select `markdown: open preview`
5. crash
I've confirmed that the issue is a bug in pulldown-cmark version 12, and
has been resolved in version
[v0.13.0](https://github.com/pulldown-cmark/pulldown-cmark/releases/tag/v0.13.0)
and specifically fixed in
pulldown-cmark/pulldown-cmark#1017
this PR simply bumps the pulldown-cmark version in zed which resolves
the crash on my local machine.
## recording
https://github.com/user-attachments/assets/dc77132f-0d43-40f3-9841-0bf34fe714fb
Release Notes:
- Fixes crash due to markdown parsing via bumping pulldown-cmark
…o stable) (#46310) Cherry-pick of #46267 to stable ---- Hi Zed team thank you for the awesome editor! I recently stumbled upon a markdown sequence that seems to cause a crash ```md - [ ] - \ - ``` *easier to read escape characters below ```rust let crash_input = "-\t[\t] -\r\\\n-" println!("{}", crash_input) ``` ## how to reproduce 1. copy the markdown above 2. save the file 3. `[shift]` + `[cmd]` + `p` to open the command palette 4. select `markdown: open preview` 5. crash I've confirmed that the issue is a bug in pulldown-cmark version 12, and has been resolved in version [v0.13.0](https://github.com/pulldown-cmark/pulldown-cmark/releases/tag/v0.13.0) and specifically fixed in pulldown-cmark/pulldown-cmark#1017 this PR simply bumps the pulldown-cmark version in zed which resolves the crash on my local machine. ## recording https://github.com/user-attachments/assets/dc77132f-0d43-40f3-9841-0bf34fe714fb Release Notes: - Fixes crash due to markdown parsing via bumping pulldown-cmark Co-authored-by: drbh <david.richard.holtz@gmail.com>
…o preview) (#46309) Cherry-pick of #46267 to preview ---- Hi Zed team thank you for the awesome editor! I recently stumbled upon a markdown sequence that seems to cause a crash ```md - [ ] - \ - ``` *easier to read escape characters below ```rust let crash_input = "-\t[\t] -\r\\\n-" println!("{}", crash_input) ``` ## how to reproduce 1. copy the markdown above 2. save the file 3. `[shift]` + `[cmd]` + `p` to open the command palette 4. select `markdown: open preview` 5. crash I've confirmed that the issue is a bug in pulldown-cmark version 12, and has been resolved in version [v0.13.0](https://github.com/pulldown-cmark/pulldown-cmark/releases/tag/v0.13.0) and specifically fixed in pulldown-cmark/pulldown-cmark#1017 this PR simply bumps the pulldown-cmark version in zed which resolves the crash on my local machine. ## recording https://github.com/user-attachments/assets/dc77132f-0d43-40f3-9841-0bf34fe714fb Release Notes: - Fixes crash due to markdown parsing via bumping pulldown-cmark Co-authored-by: drbh <david.richard.holtz@gmail.com>
Hi Zed team thank you for the awesome editor!
I recently stumbled upon a markdown sequence that seems to cause a crash
```md
- [ ] -
\
-
```
*easier to read escape characters below
```rust
let crash_input = "-\t[\t] -\r\\\n-"
println!("{}", crash_input)
```
## how to reproduce
1. copy the markdown above
2. save the file
3. `[shift]` + `[cmd]` + `p` to open the command palette
4. select `markdown: open preview`
5. crash
I've confirmed that the issue is a bug in pulldown-cmark version 12, and
has been resolved in version
[v0.13.0](https://github.com/pulldown-cmark/pulldown-cmark/releases/tag/v0.13.0)
and specifically fixed in
pulldown-cmark/pulldown-cmark#1017
this PR simply bumps the pulldown-cmark version in zed which resolves
the crash on my local machine.
## recording
https://github.com/user-attachments/assets/dc77132f-0d43-40f3-9841-0bf34fe714fb
Release Notes:
- Fixes crash due to markdown parsing via bumping pulldown-cmark
Hi Zed team thank you for the awesome editor!
I recently stumbled upon a markdown sequence that seems to cause a crash
```md
- [ ] -
\
-
```
*easier to read escape characters below
```rust
let crash_input = "-\t[\t] -\r\\\n-"
println!("{}", crash_input)
```
## how to reproduce
1. copy the markdown above
2. save the file
3. `[shift]` + `[cmd]` + `p` to open the command palette
4. select `markdown: open preview`
5. crash
I've confirmed that the issue is a bug in pulldown-cmark version 12, and
has been resolved in version
[v0.13.0](https://github.com/pulldown-cmark/pulldown-cmark/releases/tag/v0.13.0)
and specifically fixed in
pulldown-cmark/pulldown-cmark#1017
this PR simply bumps the pulldown-cmark version in zed which resolves
the crash on my local machine.
## recording
https://github.com/user-attachments/assets/dc77132f-0d43-40f3-9841-0bf34fe714fb
Release Notes:
- Fixes crash due to markdown parsing via bumping pulldown-cmark
Hi Zed team thank you for the awesome editor!
I recently stumbled upon a markdown sequence that seems to cause a crash
```md
- [ ] -
\
-
```
*easier to read escape characters below
```rust
let crash_input = "-\t[\t] -\r\\\n-"
println!("{}", crash_input)
```
## how to reproduce
1. copy the markdown above
2. save the file
3. `[shift]` + `[cmd]` + `p` to open the command palette
4. select `markdown: open preview`
5. crash
I've confirmed that the issue is a bug in pulldown-cmark version 12, and
has been resolved in version
[v0.13.0](https://github.com/pulldown-cmark/pulldown-cmark/releases/tag/v0.13.0)
and specifically fixed in
pulldown-cmark/pulldown-cmark#1017
this PR simply bumps the pulldown-cmark version in zed which resolves
the crash on my local machine.
## recording
https://github.com/user-attachments/assets/dc77132f-0d43-40f3-9841-0bf34fe714fb
Release Notes:
- Fixes crash due to markdown parsing via bumping pulldown-cmark
Hi Zed team thank you for the awesome editor!
I recently stumbled upon a markdown sequence that seems to cause a crash
```md
- [ ] -
\
-
```
*easier to read escape characters below
```rust
let crash_input = "-\t[\t] -\r\\\n-"
println!("{}", crash_input)
```
## how to reproduce
1. copy the markdown above
2. save the file
3. `[shift]` + `[cmd]` + `p` to open the command palette
4. select `markdown: open preview`
5. crash
I've confirmed that the issue is a bug in pulldown-cmark version 12, and
has been resolved in version
[v0.13.0](https://github.com/pulldown-cmark/pulldown-cmark/releases/tag/v0.13.0)
and specifically fixed in
pulldown-cmark/pulldown-cmark#1017
this PR simply bumps the pulldown-cmark version in zed which resolves
the crash on my local machine.
## recording
https://github.com/user-attachments/assets/dc77132f-0d43-40f3-9841-0bf34fe714fb
Release Notes:
- Fixes crash due to markdown parsing via bumping pulldown-cmark
Hi Zed team thank you for the awesome editor!
I recently stumbled upon a markdown sequence that seems to cause a crash
*easier to read escape characters below
how to reproduce
[shift]+[cmd]+pto open the command palettemarkdown: open previewI've confirmed that the issue is a bug in pulldown-cmark version 12, and has been resolved in version v0.13.0 and specifically fixed in pulldown-cmark/pulldown-cmark#1017
this PR simply bumps the pulldown-cmark version in zed which resolves the crash on my local machine.
recording
zedcrash.mp4
Release Notes: