Skip to content

Fix a memory leak in indexof() when evaluating an invalid expression#10916

Closed
yegappan wants to merge 1 commit intovim:masterfrom
yegappan:excmdtest
Closed

Fix a memory leak in indexof() when evaluating an invalid expression#10916
yegappan wants to merge 1 commit intovim:masterfrom
yegappan:excmdtest

Conversation

@yegappan
Copy link
Member

No description provided.

@codecov
Copy link

codecov bot commented Aug 13, 2022

Codecov Report

Merging #10916 (642885c) into master (c9b6570) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master   #10916      +/-   ##
==========================================
- Coverage   81.81%   81.81%   -0.01%     
==========================================
  Files         158      158              
  Lines      186400   186403       +3     
  Branches    42133    42133              
==========================================
+ Hits       152499   152501       +2     
- Misses      21409    21416       +7     
+ Partials    12492    12486       -6     
Flag Coverage Δ
huge-clang-none 82.68% <100.00%> (-0.01%) ⬇️
linux 82.68% <100.00%> (-0.01%) ⬇️
mingw-x64-HUGE 0.00% <0.00%> (ø)
mingw-x64-HUGE-gui 78.23% <100.00%> (+<0.01%) ⬆️
windows 77.01% <100.00%> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/evalfunc.c 90.87% <100.00%> (+<0.01%) ⬆️
src/gui_gtk_x11.c 51.81% <0.00%> (-0.19%) ⬇️
src/if_xcmdsrv.c 76.07% <0.00%> (-0.18%) ⬇️
src/channel.c 83.83% <0.00%> (-0.14%) ⬇️
src/terminal.c 77.29% <0.00%> (-0.03%) ⬇️
src/normal.c 91.32% <0.00%> (+0.03%) ⬆️
src/regexp_nfa.c 90.02% <0.00%> (+0.03%) ⬆️
src/gui.c 73.06% <0.00%> (+0.04%) ⬆️
src/os_mswin.c 49.15% <0.00%> (+0.21%) ⬆️
src/profiler.c 83.14% <0.00%> (+0.66%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@brammool brammool closed this in 63acae1 Aug 14, 2022
jamespeapen pushed a commit to jamespeapen/vim that referenced this pull request Sep 21, 2022
Problem:    indexof() may leak memory.
Solution:   Free allocated values. (Yegappan Lakshmanan, closes vim#10916)
zeertzjq added a commit to zeertzjq/neovim that referenced this pull request Feb 26, 2023
Problem:    indexof() may leak memory.
Solution:   Free allocated values. (Yegappan Lakshmanan, closes vim/vim#10916)

vim/vim@63acae1

Co-authored-by: Yegappan Lakshmanan <yegappan@yahoo.com>
zeertzjq added a commit to zeertzjq/neovim that referenced this pull request Feb 26, 2023
Problem:    indexof() may leak memory.
Solution:   Free allocated values. (Yegappan Lakshmanan, closes vim/vim#10916)

vim/vim@63acae1

Co-authored-by: Yegappan Lakshmanan <yegappan@yahoo.com>
yesean pushed a commit to yesean/neovim that referenced this pull request Mar 25, 2023
Problem:    indexof() may leak memory.
Solution:   Free allocated values. (Yegappan Lakshmanan, closes vim/vim#10916)

vim/vim@63acae1

Co-authored-by: Yegappan Lakshmanan <yegappan@yahoo.com>
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