Fixes #4426: Move r31147 to v1.8 branch (Return non-SUCCESS error codes from)

---svn-pre-commit-ignore-below---

r31147 [[BR]]
Return non-SUCCESS error codes from attribute copy functions.

Without this, an attribute copy function could return non-success, but
it would not be propagated upwards.  This caused the intel
MPI_Keyval3_* tests to fail.

cmr=v1.8:reviewer=hjelmn