Skip to content

JS: add the html argument to the jQuery functions as an XSS sink#12377

Merged
erik-krogh merged 2 commits intogithub:mainfrom
erik-krogh:jHtml
Mar 3, 2023
Merged

JS: add the html argument to the jQuery functions as an XSS sink#12377
erik-krogh merged 2 commits intogithub:mainfrom
erik-krogh:jHtml

Conversation

@erik-krogh
Copy link
Contributor

@erik-krogh erik-krogh commented Mar 3, 2023

Receognizes the sink in CVE-2023-0949

See backref for evaluation (in progress).

@github-actions github-actions bot added the JS label Mar 3, 2023
@erik-krogh erik-krogh added the no-change-note-required This PR does not need a change note label Mar 3, 2023
@erik-krogh erik-krogh marked this pull request as ready for review March 3, 2023 10:16
@erik-krogh erik-krogh requested a review from a team as a code owner March 3, 2023 10:16
@erik-krogh erik-krogh requested review from yoff and removed request for a team March 3, 2023 10:16
@asgerf
Copy link
Contributor

asgerf commented Mar 3, 2023

LGTM if the evaluation works out

Copy link
Contributor

@yoff yoff left a comment

Choose a reason for hiding this comment

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

LGTM - modulo non-problematic evaluations

@erik-krogh
Copy link
Contributor Author

Evaluation was uneventful.

@erik-krogh erik-krogh merged commit d94e51a into github:main Mar 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

JS no-change-note-required This PR does not need a change note

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants