diff options
| author | Joe Orton <jorton@apache.org> | 2004-08-11 16:25:30 +0200 |
|---|---|---|
| committer | Joe Orton <jorton@apache.org> | 2004-08-11 16:25:30 +0200 |
| commit | d2e2c4e584976c7c2e0911a3b37152aea2cd7c6c (patch) | |
| tree | e3f496b275f51822c801cf40aa8734764fb4f189 /server | |
| parent | * modules/ssl/ssl_engine_io.c (ssl_io_input_read): Fix potential (diff) | |
| download | apache2-d2e2c4e584976c7c2e0911a3b37152aea2cd7c6c.tar.xz apache2-d2e2c4e584976c7c2e0911a3b37152aea2cd7c6c.zip | |
* modules/ssl/ssl_engine_kernel.c (ssl_callback_SSLVerify_CRL),
* server/log.c (ap_log_pid),
* server/mpm/prefork/prefork.c (accept_mutex_on, accept_mutex_off),
* support/htdbm.c (htdbm_list):
Fix some non-literal format strings (warnings from gcc -Wformat-security).
PR: 30585
Submitted by: Ulf Harnhammar (SITIC), Joe Orton
git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@104548 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to 'server')
| -rw-r--r-- | server/log.c | 6 | ||||
| -rw-r--r-- | server/mpm/prefork/prefork.c | 8 |
2 files changed, 7 insertions, 7 deletions
diff --git a/server/log.c b/server/log.c index f8c2af4204..16574ab22e 100644 --- a/server/log.c +++ b/server/log.c @@ -625,9 +625,9 @@ AP_DECLARE(void) ap_log_pid(apr_pool_t *p, const char *filename) * based on the last modification time of the pid file. */ ap_log_perror(APLOG_MARK, APLOG_WARNING, 0, p, - apr_psprintf(p, "pid file %s overwritten -- Unclean " - "shutdown of previous Apache run?", - fname)); + "pid file %s overwritten -- Unclean " + "shutdown of previous Apache run?", + fname); } if ((rv = apr_file_open(&pid_file, fname, diff --git a/server/mpm/prefork/prefork.c b/server/mpm/prefork/prefork.c index 3dbbe58fff..7acd04b976 100644 --- a/server/mpm/prefork/prefork.c +++ b/server/mpm/prefork/prefork.c @@ -210,11 +210,11 @@ static void accept_mutex_on(void) if (ap_my_generation != ap_scoreboard_image->global->running_generation) { - ap_log_error(APLOG_MARK, APLOG_DEBUG, rv, NULL, msg); + ap_log_error(APLOG_MARK, APLOG_DEBUG, rv, NULL, "%s", msg); clean_child_exit(0); } else { - ap_log_error(APLOG_MARK, APLOG_EMERG, rv, NULL, msg); + ap_log_error(APLOG_MARK, APLOG_EMERG, rv, NULL, "%s", msg); exit(APEXIT_CHILDFATAL); } } @@ -228,14 +228,14 @@ static void accept_mutex_off(void) if (ap_my_generation != ap_scoreboard_image->global->running_generation) { - ap_log_error(APLOG_MARK, APLOG_DEBUG, rv, NULL, msg); + ap_log_error(APLOG_MARK, APLOG_DEBUG, rv, NULL, "%s", msg); /* don't exit here... we have a connection to * process, after which point we'll see that the * generation changed and we'll exit cleanly */ } else { - ap_log_error(APLOG_MARK, APLOG_EMERG, rv, NULL, msg); + ap_log_error(APLOG_MARK, APLOG_EMERG, rv, NULL, "%s", msg); exit(APEXIT_CHILDFATAL); } } |