You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by sf...@apache.org on 2011/10/08 09:44:11 UTC
svn commit: r1180330 - /httpd/httpd/trunk/modules/ssl/ssl_util.c
Author: sf
Date: Sat Oct 8 07:44:10 2011
New Revision: 1180330
URL: http://svn.apache.org/viewvc?rev=1180330&view=rev
Log:
Add another AP_DEBUG_ASSERT to document some assumptions in the code,
for the benefit of code analyzers.
Modified:
httpd/httpd/trunk/modules/ssl/ssl_util.c
Modified: httpd/httpd/trunk/modules/ssl/ssl_util.c
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/ssl/ssl_util.c?rev=1180330&r1=1180329&r2=1180330&view=diff
==============================================================================
--- httpd/httpd/trunk/modules/ssl/ssl_util.c (original)
+++ httpd/httpd/trunk/modules/ssl/ssl_util.c Sat Oct 8 07:44:10 2011
@@ -125,6 +125,8 @@ BOOL ssl_util_path_check(ssl_pathcheck_t
if (pcm & SSL_PCM_EXISTS && apr_stat(&finfo, path,
APR_FINFO_TYPE|APR_FINFO_SIZE, p) != 0)
return FALSE;
+ AP_DEBUG_ASSERT((pcm & SSL_PCM_EXISTS) ||
+ !(pcm & (SSL_PCM_ISREG|SSL_PCM_ISDIR|SSL_PCM_ISNONZERO)));
if (pcm & SSL_PCM_ISREG && finfo.filetype != APR_REG)
return FALSE;
if (pcm & SSL_PCM_ISDIR && finfo.filetype != APR_DIR)