[Lldb-commits] [PATCH] Fix attach API on Linux

[Greg Clayton]

Looks good.

% svn commit
Sending        source/API/SBTarget.cpp
Sending        source/Plugins/Process/POSIX/ProcessPOSIX.cpp
Sending        source/Plugins/Process/POSIX/ProcessPOSIX.h
Transmitting file data ...
Committed revision 163399.


On Sep 6, 2012, at 3:25 PM, "Kaylor, Andrew" <andrew.kaylor at intel.com> wrote:

> The attached patch fixes a problem with performing an attach from the SBTarget API on Linux (and other systems that use ProcessPOSIX).
>  
> When Process::Attach was called from SBTarget, it resulted in a call to a form of the DoAttachWithID function that wasn’t implemented in ProcessPOSIX, and so it fell back to the default implementation (which just returns an error).  It didn’t seem necessary to use the attach_info parameter for this case, so I just implemented it as a call to the simpler version of the function.
>  
> In debugging this problem, I also found that SBTarget wasn’t checking the return value from the Attach call, causing it to hang when the attach fails.
>