Mercurial > vim
diff src/message.c @ 15896:ac080f6a4db8 v8.1.0954
patch 8.1.0954: arguments of semsg() and siemsg() are not checked
commit https://github.com/vim/vim/commit/0d8562a9992e94d532485c37268ca33c0c49ecc2
Author: Bram Moolenaar <Bram@vim.org>
Date: Tue Feb 19 21:34:05 2019 +0100
patch 8.1.0954: arguments of semsg() and siemsg() are not checked
Problem: Arguments of semsg() and siemsg() are not checked.
Solution: Add function prototype with __attribute__.
| author | Bram Moolenaar <Bram@vim.org> |
|---|---|
| date | Tue, 19 Feb 2019 21:45:07 +0100 |
| parents | 7fad90423bd2 |
| children | ca0a0b64809c |
line wrap: on
line diff
--- a/src/message.c +++ b/src/message.c @@ -730,6 +730,7 @@ emsg(char *s) return TRUE; /* no error messages at the moment */ } +#ifndef PROTO // manual proto with __attribute__ /* * Print an error message with format string and variable arguments. * Note: caller must not pass 'IObuff' as 1st argument. @@ -749,6 +750,7 @@ semsg(const char *s, ...) } return TRUE; /* no error messages at the moment */ } +#endif /* * Same as emsg(...), but abort on error when ABORT_ON_INTERNAL_ERROR is @@ -765,6 +767,7 @@ iemsg(char *s) #endif } +#ifndef PROTO // manual proto with __attribute__ /* * Same as semsg(...) but abort on error when ABORT_ON_INTERNAL_ERROR is * defined. It is used for internal errors only, so that they can be @@ -783,10 +786,11 @@ siemsg(const char *s, ...) va_end(ap); emsg_core(IObuff); } -#ifdef ABORT_ON_INTERNAL_ERROR +# ifdef ABORT_ON_INTERNAL_ERROR abort(); +# endif +} #endif -} /* * Give an "Internal error" message.