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;
               }