Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SSH process breakout #868

Merged
merged 1 commit into from
Jan 25, 2017
Merged

Conversation

zachriggle
Copy link
Member

Multiple SSH enhancements

  • Break out ssh_channel into subclass ssh_process
  • Make ssh.process() directly instantiate ssh_process
  • Add ssh_process().ELF
  • Add ssh_process().corefile
  • Record the UID and GID of the executed binary
  • Add the craeted PID to the ssh_process message

@zachriggle zachriggle force-pushed the ssh-process-breakout branch 2 times, most recently from a510077 to c56b2da Compare January 24, 2017 22:22
@zachriggle zachriggle changed the title Ssh process breakout SSH process breakout Jan 24, 2017
@zachriggle zachriggle self-assigned this Jan 24, 2017
@zachriggle zachriggle added this to the Someday milestone Jan 24, 2017
@zachriggle zachriggle force-pushed the ssh-process-breakout branch 2 times, most recently from 38f828d to 18b8dee Compare January 24, 2017 23:56
- Break out ssh_channel into subclass ssh_process
- Make ssh.process() directly instantiate ssh_process
- Add ssh_process().ELF
- Add ssh_process().corefile
- Record the UID and GID of the executed binary
- Add the craeted PID to the ssh_process message
@zachriggle zachriggle merged commit 25941b5 into Gallopsled:dev Jan 25, 2017
@zachriggle zachriggle deleted the ssh-process-breakout branch January 25, 2017 00:03
@TethysSvensson TethysSvensson modified the milestones: 3.5.0, Someday Feb 12, 2017
Kyle-Kyle pushed a commit to Kyle-Kyle/pwntools that referenced this pull request Apr 25, 2021
* Fixed the failed unit testing; Improved the Dockerfile, again.

* Fixed missing new line joiner in the Dockerfile.

* Fixed the default Python version from 2 to 3.

* Moved the installation of the nasm and golang packages to the setup script.

* Fixed the extra quotes which fails the sudo check.

* Added to the Git ignore file the output files assembled and linked in the test binaries.

* Removed the output files from test binaries.

* Added to the Git ignore file the virtual environment directory.

* Added sudo to the remove and link the Python3 binary.

* Added commands related to GO, to check why is failing.

* Fixed the installation of GO, only install if not exists in Ubuntu.

* Fixed the first test built in Assembler. Checks if it is working in GitHub.

* Fixed the problem with all the assembler tests; Extracted the asserts into methods, avoiding duplicates.

* Fixed some test, should be fail the Travis tests.

* Fixed one test for Travis.

* Fixed the asserts, now find all ecpected items into the output.

* Fixed the split message, joined again.

* Fixed the last error test for loads binary.

* Fixed one missing number in the comparison.

* Fixed the first Lint errors.

* Added comment in the Dockerfile.

The commented line about `git submodule`.

pwndbg/pwndbg#868 (comment)

* Removed NASM and Go from the setup.

Created a new bash script to install them and added to the Dockerfile.

https://github.com/pwndbg/pwndbg/pull/868/files#r549075931

https://github.com/pwndbg/pwndbg/pull/868/files#r549076078

* Added the setup script in the GitHub test workflow.

https://github.com/pwndbg/pwndbg/pull/868/files#r549075931

https://github.com/pwndbg/pwndbg/pull/868/files#r549076078

* Testing the emulate cases.

It should be fail. I commented the emulation for the address 0x401000.

pwndbg/pwndbg#868 (comment)

* Fixed the different start addresses in the assembler test.

pwndbg/pwndbg#868 (comment)

pwndbg/pwndbg#868 (comment)

* Experimenting with the python location.

Commented the remove python binary and added some logs.

pwndbg/pwndbg#868 (comment)

* Removed the commands about delete Python and link the Python3.

pwndbg/pwndbg#868 (comment)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants