Skip to content

Coverity fixes#9071

Closed
slontis wants to merge 1 commit intoopenssl:masterfrom
slontis:cov_1445689
Closed

Coverity fixes#9071
slontis wants to merge 1 commit intoopenssl:masterfrom
slontis:cov_1445689

Conversation

@slontis
Copy link
Member

@slontis slontis commented Jun 4, 2019

covID 1445689 Resource leak (in error path)
covID 1445318 Resource leak (in test - minor)
covID 1443705 Unchecked return value (Needed if CRYPTO_atomic_add() was used)
covID 1443691 Resource leak (in app - minor)

Checklist
  • documentation is added or updated
  • tests are added or updated

covID 1445689 Resource leak (in error path)
covID 1445318 Resource leak (in test - minor)
covID 1443705 Unchecked return value (Needed if CRYPTO_atomic_add() was used)
covID 1443691 Resource leak (in app - minor)
@mattcaswell mattcaswell added the approval: done This pull request has the required number of approvals label Jun 4, 2019
@mspncp mspncp mentioned this pull request Jun 5, 2019
2 tasks
@slontis
Copy link
Member Author

slontis commented Jun 5, 2019

@mspncp I have no idea why this PR is being referenced from the DEVRANDOM_WAIT PR's

levitte pushed a commit that referenced this pull request Jun 6, 2019
covID 1445689 Resource leak (in error path)
covID 1445318 Resource leak (in test - minor)
covID 1443705 Unchecked return value (Needed if CRYPTO_atomic_add() was used)
covID 1443691 Resource leak (in app - minor)

Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from #9071)
@slontis slontis closed this Jun 6, 2019
@mspncp
Copy link
Contributor

mspncp commented Jun 6, 2019

Sorry, it was a typo in the Fixes #xxxx tag of that pr, which I corrected. Just ignore it.

@slontis
Copy link
Member Author

slontis commented Jun 6, 2019

thanks - I could not figure out where it was coming from :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approval: done This pull request has the required number of approvals

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants