You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by da...@apache.org on 2011/08/18 16:05:45 UTC
svn commit: r1159228 - /subversion/trunk/subversion/mod_dav_svn/liveprops.c
Author: danielsh
Date: Thu Aug 18 14:05:45 2011
New Revision: 1159228
URL: http://svn.apache.org/viewvc?rev=1159228&view=rev
Log:
For issue #2872, log (at APLOG_WARNING) invalid svn:mime-type properties being
attempted to be set. Do not add comparable logging to the other instance of
svn:mime-type validation --- in set_headers(), where those are validated for
inclusion in headers.
* subversion/mod_dav_svn/liveprops.c
(insert_prop_internal):
Log the error from svn_mime_type_validate() at APLOG_WARNING.
Modified:
subversion/trunk/subversion/mod_dav_svn/liveprops.c
Modified: subversion/trunk/subversion/mod_dav_svn/liveprops.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/mod_dav_svn/liveprops.c?rev=1159228&r1=1159227&r2=1159228&view=diff
==============================================================================
--- subversion/trunk/subversion/mod_dav_svn/liveprops.c (original)
+++ subversion/trunk/subversion/mod_dav_svn/liveprops.c Thu Aug 18 14:05:45 2011
@@ -483,6 +483,8 @@ insert_prop_internal(const dav_resource
there's no point even checking. No matter what the
error is, we can't claim to have a mime type for
this resource. */
+ ap_log_rerror(APLOG_MARK, APLOG_WARNING, serr->apr_err,
+ resource->info->r, "%s", serr->message);
svn_error_clear(serr);
return DAV_PROP_INSERT_NOTDEF;
}