diff --git a/src/smf/smfd/SmfCampaignWrapup.cc b/src/smf/smfd/SmfCampaignWrapup.cc
index a89cbffae..cc0aa03d4 100644
--- a/src/smf/smfd/SmfCampaignWrapup.cc
+++ b/src/smf/smfd/SmfCampaignWrapup.cc
@@ -221,6 +221,10 @@ bool SmfCampaignWrapup::executeCampComplete() {
   completeRollbackDn = "smfRollbackElement=CampComplete,";
   completeRollbackDn += SmfCampaignThread::instance()->campaign()->getDn();
 
+  // [Lennart] This if() is not needed. You can do all handling in the while
+  // loop and remove if((result = smfCreateRollbackElement(...
+  // Also whatever result other than SA_AIS_OK will create an ER log in syslog
+  // also if the call eventually succeeds in the while loop
   if ((result = smfCreateRollbackElement(
            completeRollbackDn,
            SmfCampaignThread::instance()->getImmHandle())) != SA_AIS_OK) {
@@ -252,6 +256,7 @@ bool SmfCampaignWrapup::executeCampComplete() {
   }
 
   for (auto& elem : m_campCompleteAction) {
+    // [Lennart] Same comment as above applies here
     if ((result = (*elem).execute(SmfCampaignThread::instance()->getImmHandle(),
                                   &completeRollbackDn)) != SA_AIS_OK) {
       LOG_ER("SmfCampaignWrapup campCompleteAction %d failed, rc=%s",