report file operation errors as warnings

The failures to change permissions and chown the ccache do not prevent
us from continuing to use this ccache. Log messages are confusing to
mod_auth_gssapi users as they leave an impression things are completely
broken.

Change log level to warning to avoid filling up logs with them on error
log level.

Signed-off-by: Alexander Bokovoy <ab@samba.org>
Reviewed-by: Simo Sorce <simo@redhat.com>

Author: abbra

src/environ.c

@@ -329,29 +329,29 @@ static void mag_set_ccname_envvar(request_rec *req, struct mag_config *cfg,
             (finfo.protection != cfg->deleg_ccache_mode)) {
             status = apr_file_perms_set(path, cfg->deleg_ccache_mode);
             if (status != APR_SUCCESS)
-                ap_log_rerror(APLOG_MARK, APLOG_ERR|APLOG_NOERRNO, 0, req,
+                ap_log_rerror(APLOG_MARK, APLOG_WARNING|APLOG_NOERRNO, 0, req,
                               "failed to set perms (%o) on file (%s)!",
                               cfg->deleg_ccache_mode, path);
         }
         if ((cfg->deleg_ccache_uid != 0) &&
             (finfo.user != cfg->deleg_ccache_uid)) {
             status = lchown(path, cfg->deleg_ccache_uid, -1);
             if (status != 0)
-                ap_log_rerror(APLOG_MARK, APLOG_ERR|APLOG_NOERRNO, 0, req,
+                ap_log_rerror(APLOG_MARK, APLOG_WARNING|APLOG_NOERRNO, 0, req,
                               "failed to set user (%u) on file (%s)!",
                               cfg->deleg_ccache_uid, path);
         }
         if ((cfg->deleg_ccache_gid != 0) &&
             (finfo.group != cfg->deleg_ccache_gid)) {
             status = lchown(path, -1, cfg->deleg_ccache_gid);
             if (status != 0)
-                ap_log_rerror(APLOG_MARK, APLOG_ERR|APLOG_NOERRNO, 0, req,
+                ap_log_rerror(APLOG_MARK, APLOG_WARNING|APLOG_NOERRNO, 0, req,
                               "failed to set group (%u) on file (%s)!",
                               cfg->deleg_ccache_gid, path);
         }
     } else {
         /* set the file cache anyway, but warn */
-        ap_log_rerror(APLOG_MARK, APLOG_ERR|APLOG_NOERRNO, 0, req,
+        ap_log_rerror(APLOG_MARK, APLOG_WARNING|APLOG_NOERRNO, 0, req,
                       "KRB5CCNAME file (%s) lookup failed!", path);
     }