We should probably clone the writer and convert failures here

[todo]

connectedhomeip/src/app/util/ember-compatibility-functions.cpp

Lines 208 to 218 in 7dbfb82

	// TODO: We should probably clone the writer and convert failures here
	// into status responses, unless our caller already does that.
	ReturnErrorOnFailure(attrOverride->Read(aClusterInfo, apWriter, &dataRead));
	if (dataRead)
	{
	// TODO: Add DataVersion support
	ReturnErrorOnFailure(
	apWriter->Put(chip::TLV::ContextTag(AttributeDataElement::kCsTag_DataVersion), kTemporaryDataVersion));
	return CHIP_NO_ERROR;
	}

---

This issue was generated by todo based on a TODO comment in 7dbfb82 when #9667 was merged. cc @bzbarsky-apple.