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.

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

Some minor fixes from coverity #6336

Merged
merged 3 commits into from Mar 31, 2020
Merged

Conversation

@moneromooo-monero
Copy link
Collaborator

moneromooo-monero commented Feb 12, 2020

No description provided.

Coverity 208373
In some contrived case, it might theoretically be the case that
destroy is called from another thread, which would modify the
threads array from two threads.

Coverity 208372
Coverity 196626
@vtnerd
vtnerd approved these changes Mar 4, 2020
@luigi1111 luigi1111 merged commit 40101e7 into monero-project:master Mar 31, 2020
4 of 5 checks passed
4 of 5 checks passed
build-macos
Details
build-windows build-windows
Details
build-ubuntu
Details
test-ubuntu
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
luigi1111 added a commit that referenced this pull request Mar 31, 2020
760ecf2 console_handler: do not let exception past the dor (moneromooo-monero)
09c8111 threadpool: lock mutex in create (moneromooo-monero)
e377977 tx_pool: catch theoretical error in get_block_reward (moneromooo-monero)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.