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

patch 9.0.1286: Coverity warns for using a NULL pointer

Browse Source 
Problem:    Coverity warns for using a NULL pointer.
Solution:   Bail out whan "varp" is NULL.
This commit is contained in:
Bram Moolenaar
2023-02-06 16:40:49 +00:00
parent a9a6b0323e
commit 546933f497
2 changed files with 13 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
src/option.c
View File

@@ -1815,7 +1815,8 @@ do_set_string(
}
/*
* Set a boolean option
* Set a boolean option.
* Returns an untranslated error message or NULL.
*/
static char *
do_set_option_bool(
@@ -1833,6 +1834,8 @@ do_set_option_bool(
if (nextchar == '=' || nextchar == ':')
return e_invalid_argument;
if (opt_idx < 0 || varp == NULL)
return NULL; // "cannot happen"
/*
* ":set opt!": invert
@@ -1870,7 +1873,8 @@ do_set_option_bool(
}
/*
* Set a numeric option
* Set a numeric option.
* Returns an untranslated error message or NULL.
*/
static char *
do_set_option_numeric(
@@ -1890,6 +1894,9 @@ do_set_option_numeric(
int i;
char *errmsg = NULL;
if (opt_idx < 0 || varp == NULL)
return NULL; // "cannot happen"
//
/*
* Different ways to set a number option:
* & set to default value
@@ -2209,8 +2216,8 @@ do_set_option(
}
/*
* allow '=' and ':' for historical reasons (MSDOS command.com
* allows only one '=' character per "set" command line. grrr. (jw)
* Allow '=' and ':' for historical reasons (MSDOS command.com).
* Allows only one '=' character per "set" command line. grrr. (jw)
*/
if (nextchar == '?'
|| (prefix == PREFIX_NONE