From e3a06e47e24f058587d169797bf1df9b4073dbd1 Mon Sep 17 00:00:00 2001
From: Mark Reynolds <mreynolds@redhat.com>
Date: Thu, 19 Oct 2017 12:20:48 -0400
Subject: [PATCH] Ticket 49374 -  server fails to start because maxdisksize is
 recognized incorrectly

Bug Description:  When directly editting dse.ldif, the server had a check
                  when setting the log maxdiskspace vs maxlogsize.  If the
                  maxlogsize is processed first nd it is higher than the
                  default maxdisksspace then it throw an error and the server
                  fails to start.

                  If you attempt this same operation using ldapmodify it
                  works as "live" updates checks all the mods first, so the
                  order of the attributes does not matter.

Fix description:  Remove the size checks from the attribute set function.
                  It is technically redundant since it is correctly checked
                  by the configdse code.

https://pagure.io/389-ds-base/issue/49374

Reviewed by: ?
---
 ldap/servers/slapd/log.c | 60 ++++++++++++------------------------------------
 1 file changed, 15 insertions(+), 45 deletions(-)

diff --git a/ldap/servers/slapd/log.c b/ldap/servers/slapd/log.c
index e16d89cc5..998efaef3 100644
--- a/ldap/servers/slapd/log.c
+++ b/ldap/servers/slapd/log.c
@@ -960,7 +960,6 @@ int
 log_set_logsize(const char *attrname, char *logsize_str, int logtype, char *returntext, int apply)
 {
     int rv = LDAP_SUCCESS;
-    PRInt64 mdiskspace = 0; /* in bytes */
     PRInt64 max_logsize;    /* in bytes */
     int logsize;            /* in megabytes */
     slapdFrontendConfig_t *fe_cfg = getFrontendConfig();
@@ -979,72 +978,43 @@ log_set_logsize(const char *attrname, char *logsize_str, int logtype, char *retu
 
     switch (logtype) {
     case SLAPD_ACCESS_LOG:
-        LOG_ACCESS_LOCK_WRITE();
-        mdiskspace = loginfo.log_access_maxdiskspace;
-        break;
-    case SLAPD_ERROR_LOG:
-        LOG_ERROR_LOCK_WRITE();
-        mdiskspace = loginfo.log_error_maxdiskspace;
-        break;
-    case SLAPD_AUDIT_LOG:
-        LOG_AUDIT_LOCK_WRITE();
-        mdiskspace = loginfo.log_audit_maxdiskspace;
-        break;
-    case SLAPD_AUDITFAIL_LOG:
-        LOG_AUDITFAIL_LOCK_WRITE();
-        mdiskspace = loginfo.log_auditfail_maxdiskspace;
-        break;
-    default:
-        PR_snprintf(returntext, SLAPI_DSE_RETURNTEXT_SIZE,
-                    "%s: invalid logtype %d", attrname, logtype);
-        rv = LDAP_OPERATIONS_ERROR;
-    }
-
-    if ((max_logsize > mdiskspace) && (mdiskspace != -1)) {
-        rv = 2;
-    }
-
-    switch (logtype) {
-    case SLAPD_ACCESS_LOG:
-        if (!rv && apply) {
+        if (apply) {
+            LOG_ACCESS_LOCK_WRITE();
             loginfo.log_access_maxlogsize = max_logsize;
             fe_cfg->accesslog_maxlogsize = logsize;
+            LOG_ACCESS_UNLOCK_WRITE();
         }
-        LOG_ACCESS_UNLOCK_WRITE();
         break;
     case SLAPD_ERROR_LOG:
-        if (!rv && apply) {
+        if (apply) {
+            LOG_ERROR_LOCK_WRITE();
             loginfo.log_error_maxlogsize = max_logsize;
             fe_cfg->errorlog_maxlogsize = logsize;
+            LOG_ERROR_UNLOCK_WRITE();
         }
-        LOG_ERROR_UNLOCK_WRITE();
         break;
     case SLAPD_AUDIT_LOG:
-        if (!rv && apply) {
+        if (apply) {
+            LOG_AUDIT_LOCK_WRITE();
             loginfo.log_audit_maxlogsize = max_logsize;
             fe_cfg->auditlog_maxlogsize = logsize;
+            LOG_AUDIT_UNLOCK_WRITE();
         }
-        LOG_AUDIT_UNLOCK_WRITE();
         break;
     case SLAPD_AUDITFAIL_LOG:
-        if (!rv && apply) {
+        if (apply) {
+            LOG_AUDITFAIL_LOCK_WRITE();
             loginfo.log_auditfail_maxlogsize = max_logsize;
             fe_cfg->auditfaillog_maxlogsize = logsize;
+            LOG_AUDITFAIL_UNLOCK_WRITE();
         }
-        LOG_AUDITFAIL_UNLOCK_WRITE();
         break;
     default:
-        rv = 1;
-    }
-    /* logsize is in MB */
-    if (rv == 2) {
-        slapi_log_err(SLAPI_LOG_ERR, "log_set_logsize",
-                      "Invalid value for Maximum log size:"
-                      "Maxlogsize:%d (MB) exceeds Maxdisksize:%ld (MB)\n",
-                      logsize, (long int)(mdiskspace / LOG_MB_IN_BYTES));
-
+        PR_snprintf(returntext, SLAPI_DSE_RETURNTEXT_SIZE,
+                    "%s: invalid logtype %d", attrname, logtype);
         rv = LDAP_OPERATIONS_ERROR;
     }
+
     return rv;
 }
 
-- 
2.13.6