You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by ja...@apache.org on 2015/11/29 17:06:29 UTC
svn commit: r1717086 - /httpd/httpd/trunk/modules/arch/win32/mod_isapi.c
Author: jailletc36
Date: Sun Nov 29 16:06:28 2015
New Revision: 1717086
URL: http://svn.apache.org/viewvc?rev=1717086&view=rev
Log:
Silent a sparse Warning.
cid->ecb->lpszLogData can not be NULL.
It is defined in mod_isapi.h as:
typedef struct EXTENSION_CONTROL_BLOCK {
...
char lpszLogData[80];
...
Modified:
httpd/httpd/trunk/modules/arch/win32/mod_isapi.c
Modified: httpd/httpd/trunk/modules/arch/win32/mod_isapi.c
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/arch/win32/mod_isapi.c?rev=1717086&r1=1717085&r2=1717086&view=diff
==============================================================================
--- httpd/httpd/trunk/modules/arch/win32/mod_isapi.c (original)
+++ httpd/httpd/trunk/modules/arch/win32/mod_isapi.c Sun Nov 29 16:06:28 2015
@@ -1582,9 +1582,10 @@ static apr_status_t isapi_handler (reque
rv = (*isa->HttpExtensionProc)(cid->ecb);
/* Check for a log message - and log it */
- if (cid->ecb->lpszLogData && *cid->ecb->lpszLogData)
+ if (*cid->ecb->lpszLogData) {
ap_log_rerror(APLOG_MARK, APLOG_INFO, 0, r, APLOGNO(02113)
"%s: %s", r->filename, cid->ecb->lpszLogData);
+ }
switch(rv) {
case 0: /* Strange, but MS isapi accepts this as success */