gdb attaching to process during tests has non-sudo solution #15244

pull instagibbs wants to merge 1 commits into bitcoin:master from instagibbs:gdb_ptrace changing 1 files +2 −1

instagibbs commented at 6:10 PM on January 24, 2019: member

There are some security considerations, so a link is attached.
instagibbs commented at 6:10 PM on January 24, 2019: member

I'm not sure what the policy is about linking static web-pages for informational purposes.
fanquake added the label Docs on Jan 24, 2019
gdb attaching to process during tests has non-sudo solution f96dbd1bbe

in test/README.md:178 in 04b5513107 outdated

 174 | @@ -175,7 +175,7 @@ cat /tmp/user/1000/testo9vsdjo3/node1/regtest/bitcoind.pid
 175 |  gdb /home/example/bitcoind <pid>
 176 |  ```
 177 |  
 178 | -Note: gdb attach step may require `sudo`
 179 | +Note: gdb attach step may require ptrace_scope to be set to 0, or sudo. See this link for considerations: https://www.kernel.org/doc/Documentation/security/Yama.txt

laanwj commented at 12:16 PM on January 31, 2019:

Please keep the quotes

instagibbs commented at 12:25 PM on January 31, 2019:

fixed.(I think)

instagibbs force-pushed on Jan 31, 2019
fanquake commented at 12:35 PM on January 31, 2019: member

utACK f96dbd1
laanwj commented at 12:50 PM on January 31, 2019: member

utACK (but I'm aware this solution exists and works and know this is correct) f96dbd1bbeeea82f07bc71c695fb17e8d7c9f1aa
laanwj merged this on Jan 31, 2019
laanwj closed this on Jan 31, 2019
laanwj referenced this in commit 4d661baf1a on Jan 31, 2019
PastaPastaPasta referenced this in commit 8c0f5f5621 on Jun 27, 2021
PastaPastaPasta referenced this in commit 9390a6b32e on Jun 28, 2021
PastaPastaPasta referenced this in commit 4085a47dc6 on Jun 28, 2021
PastaPastaPasta referenced this in commit 63272f5b94 on Jun 29, 2021
DrahtBot locked this on Dec 16, 2021

Contributors

Labels