Skip to content

Conversation

geoffw0
Copy link
Contributor

@geoffw0 geoffw0 commented Oct 8, 2018

This is based on #224 and expands the fix to cover all of our CWE-119 and CWE-120 queries.

@geoffw0 geoffw0 added the C++ label Oct 8, 2018
@jbj jbj added the depends on internal PR This PR should only be merged in sync with an internal Semmle PR label Oct 8, 2018
@geoffw0
Copy link
Contributor Author

geoffw0 commented Oct 12, 2018

The tests pass with the .expected changes in https://git.semmle.com/Semmle/code/pull/28315.

@geoffw0 geoffw0 force-pushed the zerosizebuffer branch 2 times, most recently from fc66574 to c6fc836 Compare October 29, 2018 10:22
@geoffw0
Copy link
Contributor Author

geoffw0 commented Oct 29, 2018

Rebased, merge fixed.

@geoffw0 geoffw0 requested a review from a team as a code owner November 6, 2018 14:27
@jbj jbj merged commit e03b4f0 into github:master Nov 6, 2018
aibaars pushed a commit that referenced this pull request Oct 14, 2021
smowton pushed a commit to smowton/codeql that referenced this pull request Apr 16, 2022
Kotlin: Comments and tweaks in Label
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C++ depends on internal PR This PR should only be merged in sync with an internal Semmle PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants