thac0/vim
1
0
Fork 0
forked from aniani/vim
Code Activity

patch 9.0.1405: missing check for out-of-memory

Browse Source 
Problem:    Missing check for out-of-memory.
Solution:   Check for alloc() returning NULL pointer. (closes #12149)
This commit is contained in:
Bram Moolenaar
2023-03-15 22:05:44 +00:00
parent e638acc927
commit 14338024c1
2 changed files with 18 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

30
src/option.c
View File

@@ -126,15 +126,14 @@ set_init_default_backupskip(void)
#endif
int len;
garray_T ga;
int mustfree;
char_u *item;
char_u *item;
opt_idx = findoption((char_u *)"backupskip");
ga_init2(&ga, 1, 100);
for (n = 0; n < (long)ARRAY_LENGTH(names); ++n)
{
mustfree = FALSE;
int mustfree = FALSE;
#ifdef UNIX
if (*names[n] == NUL)
# ifdef MACOS_X
@@ -150,19 +149,22 @@ set_init_default_backupskip(void)
// First time count the NUL, otherwise count the ','.
len = (int)STRLEN(p) + 3;
item = alloc(len);
STRCPY(item, p);
add_pathsep(item);
STRCAT(item, "*");
if (find_dup_item(ga.ga_data, item, options[opt_idx].flags)
== NULL
&& ga_grow(&ga, len) == OK)
if (item != NULL)
{
if (ga.ga_len > 0)
STRCAT(ga.ga_data, ",");
STRCAT(ga.ga_data, item);
ga.ga_len += len;
STRCPY(item, p);
add_pathsep(item);
STRCAT(item, "*");
if (find_dup_item(ga.ga_data, item, options[opt_idx].flags)
== NULL
&& ga_grow(&ga, len) == OK)
{
if (ga.ga_len > 0)
STRCAT(ga.ga_data, ",");
STRCAT(ga.ga_data, item);
ga.ga_len += len;
}
vim_free(item);
}
vim_free(item);
}
if (mustfree)
vim_free(p);