diff --git a/src/log/logd/lgs_config.cc b/src/log/logd/lgs_config.cc
index c1549a9d9..cc8fac94d 100644
--- a/src/log/logd/lgs_config.cc
+++ b/src/log/logd/lgs_config.cc
@@ -506,9 +506,20 @@ done:
  * - Length of root path <= PATH_MAX
  * - Root path must be a direcory where log can create files and write
  *
+ * [Lennart] the new errno_save parameter must be described:
+ * - Is an [out] parameter
+ * - Must be a valid poiner, NULL pointer is not allowed
+ * - What values may be returned?
  * @param root_str_in[in] Root path to verify
  * @return -1 on error
  */
+// [Lennart] In general this is not a good change. To add an out parameter
+// breaks the consistency of keeping the usage of the validation functions
+// simple by limiting parameter handling to just input the parameter(s) to be
+// validated and just give a Valid or NotValid answer.
+// In practice the validation has failed regardless if it is because of an
+// invalid parameter or if it could not be verified. Also see my comment in the
+// lgs_imm.cc file
 int lgs_cfg_verify_root_dir(const std::string &root_str_in, int *errno_save) {
   int rc = 0;
   log_stream_t *stream = nullptr;
diff --git a/src/log/logd/lgs_imm.cc b/src/log/logd/lgs_imm.cc
index d61a12fba..3748de0f0 100644
--- a/src/log/logd/lgs_imm.cc
+++ b/src/log/logd/lgs_imm.cc
@@ -780,6 +780,15 @@ static SaAisErrorT config_ccb_completed_modify(
             report_oi_error(immOiHandle, opdata->ccbId, "Log server is BUSY");
             ais_rc = SA_AIS_ERR_NO_RESOURCES;
           } else {
+            // [Lennart] It may be valid to use different return codes here
+            // since SA_AIS_ERR_NO_RESOURCES means that the CCB may have been
+            // correct. In this case the parameter may be incorrect since the
+            // missing "resource" prevents a verification so is this really
+            // meaningful? Also to "try again" the user must recreate the whole
+            // CCB. It is not possible to just try CCB apply again.
+            // However it is relevant to change the error message to something
+            // like:
+            // "pathName: %s is NOT accepted OR could not be verified"
             report_oi_error(immOiHandle, opdata->ccbId,
                             "pathName: %s is NOT accepted", pathName.c_str());
             ais_rc = SA_AIS_ERR_BAD_OPERATION;