Skip to content

Commit

Permalink
Test BSS scanning code
Browse files Browse the repository at this point in the history
Seems like we can't figure out the python interpreter address w/o symbols,
which definitely used to work . Testing this out in CI to figure out what
version this started to fail at
  • Loading branch information
benfred committed Oct 30, 2024
1 parent 6eef487 commit d9b51a2
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion src/python_process_info.rs
Original file line number Diff line number Diff line change
Expand Up @@ -354,7 +354,10 @@ where

// Make sure the interpreter addr is valid before returning
match check_interpreter_addresses(&[addr], &*python_info.maps, process, version) {
Ok(addr) => return Ok(addr),
Ok(addr) => {
warn!("skipping symbol interpreter to test BSS scanning code {:016x}", addr);
// return Ok(addr),
}
Err(_) => {
warn!(
"Interpreter address from _PyRuntime symbol is invalid {:016x}",
Expand Down

0 comments on commit d9b51a2

Please sign in to comment.