address comments

Author: yunhanw-google

src/controller/CHIPDeviceController.cpp

@@ -2376,6 +2376,9 @@ CHIP_ERROR DeviceCommissioner::ParseICDInfo(ReadCommissioningInfo & info)
 
     if (!isICD)
     {
+        info.icd.idleModeDuration = 0;
+        info.icd.activeModeDuration = 0;
+        info.icd.activeModeThreshold = 0;
         return CHIP_NO_ERROR;
     }
 
@@ -2389,15 +2392,15 @@ CHIP_ERROR DeviceCommissioner::ParseICDInfo(ReadCommissioningInfo & info)
         mAttributeCache->Get<IcdManagement::Attributes::ActiveModeDuration::TypeInfo>(kRootEndpointId, info.icd.activeModeDuration);
     if (err != CHIP_NO_ERROR)
     {
-        ChipLogError(Controller, "IcdManagement.ActiveModeDuration expected, but failed to read.");
+        ChipLogError(Controller, "IcdManagement.ActiveModeDuration expected, but failed to read: %" CHIP_ERROR_FORMAT, err.Format());
         return err;
     }
 
     err = mAttributeCache->Get<IcdManagement::Attributes::ActiveModeThreshold::TypeInfo>(kRootEndpointId,
                                                                                          info.icd.activeModeThreshold);
     if (err != CHIP_NO_ERROR)
     {
-        ChipLogError(Controller, "IcdManagement.ActiveModeThreshold expected, but failed to read.");
+        ChipLogError(Controller, "IcdManagement.ActiveModeThreshold expected, but failed to read: %" CHIP_ERROR_FORMAT, err.Format());
     }
 
     return err;