prov/shm: HMEM async copy path does double copy

[Zach Dworkin]

Describe the bug
In smr_progres_ipc if there is an async copy requested the function will do a smr_ipc_async_copy call and then fall into the code below where it does either a ofi_copy_from_hmem_iov or an ofi_copy_to_hmem_iov. This should probably just return smr_ipc_async_copy instead of save the ret.

To Reproduce
Found while diagnosing error paths for a new protocol

Expected behavior
Just do the one copy and then close the fd on the other side

Output
Found while diagnosing error paths for a new protocol

Environment:
N/A

Additional context
None

[aingerson]

Issue not seen upstream, false report, my fault!