id	summary	reporter	owner	description	type	status	priority	milestone	component	version	severity	resolution	keywords	cc
836	psMetadataCopy - partially copied metadata	David.Robbins@…	Paul Price	"In addition to the concerns I've expressed about psMetadataCopy not freeing
""out"" and setting it to NULL on error, I also realized that a psMetadata could
be partially copied before erroring and returning NULL.  ie, a metadataItem in
""in"" with an invalid type will cause NULL to be returned, but will not return
the original-unaltered ""out"" either.  
We're suggesting two things be done to psMetadataCopy.  First change it to:
bool psMetadataCopy(psMetadata **out, const psMetadata *in);
second, either skip the bad psMetadataItem instead of erroring-out, or keep
track of the number of items added and remove each one to return the original ""out""."	defect	closed	high		types	unspecified	normal	invalid