Skip to content

Coverity fixes and leak fix - #17189

Closed
chrisbra wants to merge 4 commits into
vim:masterfrom
chrisbra:coverity
Closed

chrisbra wants to merge 4 commits into
vim:masterfrom
chrisbra:coverity

Conversation

@chrisbra

Copy link
Copy Markdown
Member

No description provided.

Fixes: Coverity issue: 1646573
tmp.v_lock is left unitialized

Signed-off-by: Christian Brabandt <[email protected]>
Fixes: Coverity Issue: 1646574
explicitly cast return value to (void)

Signed-off-by: Christian Brabandt <[email protected]>
Fixes: Coverity issue 1646601
verify that ccline is not null, before dereferencing it

Signed-off-by: Christian Brabandt <[email protected]>
Problem:  cmdcomplete_info() leaks memory
Solution: free cmdline_orig before assigning new memory to it

Signed-off-by: Christian Brabandt <[email protected]>
chrisbra added a commit that referenced this pull request Apr 22, 2025
Problem:  Coverity: complains about unutilized variable
Solution: initialize typval properly
          (author)

Fixes: Coverity issue: 1646573
       tmp.v_lock is left unitialized
related: #17189

Signed-off-by: Christian Brabandt <[email protected]>
chrisbra added a commit that referenced this pull request Apr 22, 2025
Problem:  Coverity complains about unchecked return value
Solution: cast return value to (void)

Fixes: Coverity issue 1646574
related: #17189

Signed-off-by: Christian Brabandt <[email protected]>
@chrisbra chrisbra closed this in 362be6b Apr 22, 2025
@chrisbra
chrisbra deleted the coverity branch September 10, 2025 05:50
janlazo added a commit to janlazo/neovim that referenced this pull request Jun 16, 2026
Problem:  Coverity complains about Null pointer dereferences
Solution: before accessing ccline->cmdbuff check that ccline is not NULL

Fixes: Coverity issue 1646601
closes: vim/vim#17189

vim/vim@362be6b

Co-authored-by: Christian Brabandt <[email protected]>
justinmk pushed a commit to neovim/neovim that referenced this pull request Jun 16, 2026
…#40280

Problem:  Coverity complains about Null pointer dereferences
Solution: before accessing ccline->cmdbuff check that ccline is not NULL

Fixes: Coverity issue 1646601
closes: vim/vim#17189

vim/vim@362be6b

Co-authored-by: Christian Brabandt <[email protected]>
barrettruth pushed a commit to barrettruth/neovim that referenced this pull request Jun 23, 2026
…neovim#40280

Problem:  Coverity complains about Null pointer dereferences
Solution: before accessing ccline->cmdbuff check that ccline is not NULL

Fixes: Coverity issue 1646601
closes: vim/vim#17189

vim/vim@362be6b

Co-authored-by: Christian Brabandt <[email protected]>
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.

1 participant