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

Problem: use of malloc is error-prone #3573

Merged
merged 1 commit into from
Jul 10, 2019

Conversation

sigiesec
Copy link
Member

Solution: use std::vector instead, simplify control-flow

Solution: use std::vector instead, simplify control-flow
@sonatype-lift
Copy link

sonatype-lift bot commented Jul 10, 2019

Persistent Bugs or Warnings (24)

Infer

  1. DEAD_STORE
  2. DEAD_STORE
  3. DEAD_STORE
  4. DEAD_STORE
  5. MEMORY_LEAK
    • At src/radix_tree.cpp(407,1)
    • memory dynamically allocated by call to make_node() at line 340, column 11 is not reachable after line 407, column 1.
  6. MEMORY_LEAK
    • At src/radix_tree.cpp(407,1)
    • memory dynamically allocated to current_node._data by call to resize at line 356, column 9 is not reachable after line 407, column 1.
  7. MEMORY_LEAK
    • At src/radix_tree.cpp(527,1)
    • memory dynamically allocated to current_node._data by call to resize at line 448, column 9 is not reachable after line 527, column 1.
  8. MEMORY_LEAK
    • At src/radix_tree.cpp(527,1)
    • memory dynamically allocated to parent_node._data by call to resize at line 477, column 9 is not reachable after line 527, column 1.
  9. UNINITIALIZED_VALUE
  10. UNINITIALIZED_VALUE
  11. UNINITIALIZED_VALUE
  12. UNINITIALIZED_VALUE
  13. UNINITIALIZED_VALUE
  14. UNINITIALIZED_VALUE
  15. UNINITIALIZED_VALUE

@sonatype-lift
Copy link

sonatype-lift bot commented Jul 10, 2019

Muse Analysis Complete

Analysis tools: [ Infer ] run on commits [ 1de4cf6 .. b5db4b4 ]

@somdoron somdoron merged commit 9c5cf85 into zeromq:master Jul 10, 2019
@sigiesec sigiesec deleted the use-std-vector branch July 11, 2019 10:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants