Skip to content
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.

Sign up for GitHub

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

Jump to bottom

サブマッチに対応する #8

Open
yapatta opened this issue Nov 21, 2020 · 2 comments · Fixed by #33
Open

サブマッチに対応する #8

yapatta opened this issue Nov 21, 2020 · 2 comments · Fixed by #33
Assignees
@masa5555

Comments

@yapatta
Copy link
Collaborator

yapatta commented Nov 21, 2020

/^ $/以外の部分マッチの時先頭は.*?, 末尾は.*にしておく。
先頭はnon-greedyでマッチさせたい(本来の文字の探索が重要だから)。
@yapatta
Copy link
Collaborator Author

yapatta commented Nov 21, 2020

今現在のタスクが終わり次第取り組みます。
難しそうならヘルプを頼みます。

@masa5555 masa5555 self-assigned this Nov 28, 2020
@masa5555
Copy link
Collaborator

masa5555 commented Nov 28, 2020 •
edited
Loading

/^ $/以外の部分マッチの時先頭は.*?, 末尾は.*にしておく。
先頭はnon-greedyでマッチさせたい(本来の文字の探索が重要だから)。

加えて
^, $が正しい位置にない場合は、例外を投げる

masa5555 added a commit that referenced this issue Nov 28, 2020
@masa5555
submatch対応 #8
3e9c5c0
@masa5555 masa5555 linked a pull request Nov 28, 2020 that will close this issue
submatch対応 #8 #33
Merged
@masa5555 masa5555 removed a link to a pull request Nov 30, 2020
submatch対応 #8 #33
Merged
@masa5555 masa5555 linked a pull request Nov 30, 2020 that will close this issue
submatch対応 #8 #33
Merged
yapatta added a commit that referenced this issue Dec 7, 2020
@yapatta
Merge pull request #33 from n4o847/submatch
7d2787a 
submatch対応 #8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@masa5555 masa5555

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

submatch対応 #8 n4o847/seccamp-redos
2 participants
@yapatta @masa5555