From 546933f4974d0e0966a4d4e78a6a46b7532d71c6 Mon Sep 17 00:00:00 2001 From: Bram Moolenaar Date: Mon, 6 Feb 2023 16:40:49 +0000 Subject: patch 9.0.1286: Coverity warns for using a NULL pointer Problem: Coverity warns for using a NULL pointer. Solution: Bail out whan "varp" is NULL. --- src/option.c | 15 +++++++++++---- src/version.c | 2 ++ 2 files changed, 13 insertions(+), 4 deletions(-) diff --git a/src/option.c b/src/option.c index 420d36d05..66d681e48 100644 --- a/src/option.c +++ b/src/option.c @@ -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 diff --git a/src/version.c b/src/version.c index 7c85a97e6..439698b32 100644 --- a/src/version.c +++ b/src/version.c @@ -695,6 +695,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ +/**/ + 1286, /**/ 1285, /**/ -- cgit v1.2.1