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

Greg Clayton gclayton at apple.com
Fri Sep 7 10:53:19 PDT 2012


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.
> 





More information about the lldb-commits mailing list