Defect #32563
closedRedmine 4 crashing with SEGFAULT under stress test when Markdown is used
0%
Description
Disclosure: my real motivation is to provide even more ammunition for #32424. But this issue is still valid and represents also a DOS and possibly other security vulnerability.
The Redcarpet-based wiki formatter is shared in Redmine, see source:trunk/lib/redmine/wiki_formatting/markdown/formatter.rb. I guess that with Rails 5 on Redmine 4, multithreaded operation became available by default. And when the app server supports multithreading, it just happens that Redmine is multithreaded (not a Rails expert). Redcarpet instance is not thread-safe - I've found this issue, opened for more than three years atm.
Steps to reproduce:docker run redmine4
Note1- Configure markdown as rext formatting, create some larger wiki pages, create an API key
- Run a stress test with parallel requests, we were using 12 workers invoking
curl -sf -g -H "X-Redmine-API-Key: $api_key" -o "$o" "$url/$q"
1 Yes, it's using not recommended Webrick within rails server
, still hope it's not an excuse for this behavior. :) Actually, we first came across this when creating a rake task for processing markup format conversions in parallel. But it happened also when we were doing rendering tests using standard Redmine stack, which is this issue.
Expected result: everything is rendering fine.
Actual result: ruby segfaults after a few hundred pages rendered.
ruby: markdown.c:2896: sd_markdown_render: Assertion `md->work_bufs[BUFFER_SPAN].size == 0' failed. /usr/src/redmine/lib/redmine/wiki_formatting/markdown/formatter.rb:82: [BUG] Segmentation fault at 0x0000000000000000 ruby 2.6.5p114 (2019-10-01 revision 67812) [x86_64-linux]Possible solutions:
- Do not share Redcarpet formatter
- Mutex it
- Make it thread local
- Document that multithreaded operation must be prevented when Markdown is used
- Get rid of Redcarpet (yes, please! Plus #32424)
Related issues
Updated by Go MAEDA almost 5 years ago
- Related to Feature #32424: CommonMark Markdown Text Formatting added
Updated by Marius BĂLTEANU over 3 years ago
- Status changed from New to Closed
- Resolution set to Wont fix
The current Markdown implementation based on RedCarpet is going to be dropped in the future versions and it will be replaced by the CommonMark Markdown (Github Flavoured) formatter that was just committed for 5.0.0.
Updated by Holger Just 10 months ago
- Has duplicate Defect #40131: markdown/formatter.rb:81: [BUG] Segmentation fault at 0x0000000000000000 added