fix a serious bug in libproc's proc_attach

proc_attach always frees any struct proc_handle data
that it allocates, but that is supposed to be done
only in error conditions.

PR: bin/158431
Approved by: re (kib)
MFC after: 1 week