If domain started, maybe destroy it first before sync is better. Undefine then redefine domain while domain is running is ok, the seclabel setting might be different between the running domain and original persistent setting, following case with the domain running will not get right xml definition when do live dumpxml. 
If all cases first kill domain then this is not a problem and could be ignored.