Author: jmolenda
Date: Thu Oct 16 03:43:27 2014
New Revision: 219920
URL: http://llvm.org/viewvc/llvm-project?rev=219920&view=rev
Log:
Most of this function checks to see if m_process is non-null before
dereferencing it, except for this one section of code. Add a null
check around it.
clang static analyzer fix.
Modified:
lldb/trunk/source/Plugins/DynamicLoader/POSIX-DYLD/DynamicLoaderPOSIXDYLD.cpp
Modified:
lldb/trunk/source/Plugins/DynamicLoader/POSIX-DYLD/DynamicLoaderPOSIXDYLD.cpp
URL:
http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Plugins/DynamicLoader/POSIX-DYLD/DynamicLoaderPOSIXDYLD.cpp?rev=219920&r1=219919&r2=219920&view=diff
==============================================================================
---
lldb/trunk/source/Plugins/DynamicLoader/POSIX-DYLD/DynamicLoaderPOSIXDYLD.cpp
(original)
+++
lldb/trunk/source/Plugins/DynamicLoader/POSIX-DYLD/DynamicLoaderPOSIXDYLD.cpp
Thu Oct 16 03:43:27 2014
@@ -257,13 +257,15 @@ DynamicLoaderPOSIXDYLD::ProbeEntry()
if (log)
log->Printf ("DynamicLoaderPOSIXDYLD::%s pid %" PRIu64 "
GetEntryPoint() returned address 0x%" PRIx64 ", setting entry breakpoint",
__FUNCTION__, m_process ? m_process->GetID () : LLDB_INVALID_PROCESS_ID, entry);
+ if (m_process)
+ {
+ Breakpoint *const entry_break =
m_process->GetTarget().CreateBreakpoint(entry, true, false).get();
+ entry_break->SetCallback(EntryBreakpointHit, this, true);
+ entry_break->SetBreakpointKind("shared-library-event");
- Breakpoint *const entry_break =
m_process->GetTarget().CreateBreakpoint(entry, true, false).get();
- entry_break->SetCallback(EntryBreakpointHit, this, true);
- entry_break->SetBreakpointKind("shared-library-event");
-
- // Shoudn't hit this more than once.
- entry_break->SetOneShot (true);
+ // Shoudn't hit this more than once.
+ entry_break->SetOneShot (true);
+ }
}
// The runtime linker has run and initialized the rendezvous structure once the
_______________________________________________
lldb-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits