emberAfAttributeChanged callback invoked twice for endpoint 0 's PartsList when endpoint is added

[chialunl]

When adding an endpoint, emberAfEndpointEnableDisable will be called twice, Once

connectedhomeip/src/app/util/attribute-storage.cpp

Line 939 in d5e9829

emberAfAttributeChanged(parentEndpointId, Clusters::Descriptor::Id, Clusters::Descriptor::Attributes::PartsList::Id,

and another time

connectedhomeip/src/app/util/attribute-storage.cpp

Line 950 in d5e9829

emberAfAttributeChanged(/* endpoint = */ 0, Clusters::Descriptor::Id, Clusters::Descriptor::Attributes::PartsList::Id,

[bzbarsky-apple]

Why is this a problem @chialunl? emberAfAttributeChanged just marks the attribute dirty. That's an idempotent operation, so calling it multiple times in a row for the same attribute is fine.

[bzbarsky-apple]

But also, parentEndpointId shouldn't ever be 0 in that loop, I would think (unless someone has mis-configured their endpoints: parentEndpointId is for listing the parent endpoint if it's not the root node endpoint), so this shouldn't be an issue in practice anyway.