Correct memory double-free of __pmGetAttrValue result
Correct memory double-free of __pmGetAttrValue result

__pmGetAttrValue doesn't strdup its return value (and it's right
not to waste time that way).  Its caller __pmAuthSecretCB thus
shouldn't free() it.  (It'll be freed eventually during pcp-level
context-attribute cleanup.)

edwardt/pcp,wuliming/pcp,edwardt/pcp,aeg-aeg/pcpfans,prasincs/pcp,aeg-aeg/pcpfans,tjanez/pcp,edwardt/pcp,prasincs/pcp,tjanez/pcp,edwardt/pcp,adfernandes/pcp,adfernandes/pcp,andyvand/cygpcpfans,wuliming/pcp,wuliming/pcp,tjanez/pcp,adfernandes/pcp,prasincs/pcp,adfernandes/pcp,wuliming/pcp,andyvand/cygpcpfans,andyvand/cygpcpfans,tjanez/pcp,aeg-aeg/pcpfans,andyvand/cygpcpfans,prasincs/pcp,prasincs/pcp,andyvand/cygpcpfans,aeg-aeg/pcpfans,aeg-aeg/pcpfans,edwardt/pcp,aeg-aeg/pcpfans,tjanez/pcp,prasincs/pcp,andyvand/cygpcpfans,edwardt/pcp,andyvand/cygpcpfans,tjanez/pcp,adfernandes/pcp,wuliming/pcp,prasincs/pcp,edwardt/pcp,adfernandes/pcp,wuliming/pcp,adfernandes/pcp,edwardt/pcp,adfernandes/pcp,aeg-aeg/pcpfans,tjanez/pcp,aeg-aeg/pcpfans,wuliming/pcp,tjanez/pcp,andyvand/cygpcpfans,wuliming/pcp,prasincs/pcp