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

Merge pull request NixOS#488 from cedric84/patch-1 #11

Merged
merged 64 commits into from
Mar 27, 2023

Conversation

fengjixuchui
Copy link
Owner

Thank you!

Please do your best to include a regression test
so that the quality of future releases can be preserved.

cgzones and others added 30 commits January 28, 2023 23:27
Add options the modify the state of the executable flag of the GNU_STACK
program header. That header indicates whether the object is requiring an
executable stack.
Bumps [cachix/install-nix-action](https://github.com/cachix/install-nix-action) from 18 to 19.
- [Release notes](https://github.com/cachix/install-nix-action/releases)
- [Commits](cachix/install-nix-action@v18...v19)

---
updated-dependencies:
- dependency-name: cachix/install-nix-action
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
This patch checks if the section header table is placed right after the
program header table such that it would overlap when we add a new entry
in the program header table.
If that is the case, move the section header table to the end of the file.

Moreover, there is no need to add a new PT_LOAD segment everytime.
Check if the last segment is already a PT_LOAD with the same characteristics
and adjacent. Extend it in this case.
458: Bump cachix/install-nix-action from 18 to 19 r=Mic92 a=dependabot[bot]



Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Jörg Thalheim <Mic92@users.noreply.github.com>
456: Add options to print, clear and set executable stack state r=Mic92 a=cgzones



Co-authored-by: Christian Göttsche <cgzones@googlemail.com>
463: Apply fixes from shellcheck r=Mic92 a=Mic92



Co-authored-by: Jörg Thalheim <joerg@thalheim.io>
Co-authored-by: Jörg Thalheim <Mic92@users.noreply.github.com>
Enable to parse the header file on its own, e.g. for language servers
(clangd).
Make the behavior of functions more explicit.
Avoid creating a stringstream by using find().
Also declare the function static and warn if return value not used.
Prevent overflows in the addtion of q and size, and avoid truncations in
callers by using size_t as type for size.
bors bot and others added 29 commits February 21, 2023 20:01
464: Modernizations and strictness improvements r=Mic92 a=cgzones



Co-authored-by: Christian Göttsche <cgzones@googlemail.com>
Co-authored-by: Jörg Thalheim <Mic92@users.noreply.github.com>
Co-authored-by: Jörg Thalheim <Mic92@users.noreply.github.com>
Co-authored-by: Jörg Thalheim <Mic92@users.noreply.github.com>
Untested. Coding from github while my VM is acting up.
Round up 0 to m instead of wrapping around and return an unexpected
result, which is not a multiple of m.
459: Add feature to rename dynamic symbols r=Mic92 a=brenoguim



Co-authored-by: Breno Rodrigues Guimaraes <brenorg@gmail.com>
Co-authored-by: Breno Rodrigues Guimarães <brenorg@gmail.com>
Co-authored-by: Jörg Thalheim <Mic92@users.noreply.github.com>
466: Adjust roundUp for 0 as input r=Mic92 a=cgzones



Co-authored-by: Christian Göttsche <cgzones@googlemail.com>
Co-authored-by: Jörg Thalheim <Mic92@users.noreply.github.com>
460: Avoid overlapping program header table with section header table #457 r=Mic92 a=brenoguim



Co-authored-by: Breno Rodrigues Guimaraes <brenorg@gmail.com>
Co-authored-by: Breno Rodrigues Guimarães <brenorg@gmail.com>
Co-authored-by: Jörg Thalheim <Mic92@users.noreply.github.com>
Bumps [cachix/install-nix-action](https://github.com/cachix/install-nix-action) from 19 to 20.
- [Release notes](https://github.com/cachix/install-nix-action/releases)
- [Commits](cachix/install-nix-action@v19...v20)

---
updated-dependencies:
- dependency-name: cachix/install-nix-action
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
476: Bump cachix/install-nix-action from 19 to 20 r=Mic92 a=dependabot[bot]



Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
473: Other switches might set changed as true. Use extraStrings size. #416 r=Mic92 a=brenoguim



Co-authored-by: Breno Rodrigues Guimaraes <brenorg@gmail.com>
Co-authored-by: Jörg Thalheim <Mic92@users.noreply.github.com>
Use the largest segment alignment for libraries requiring non-standard alignments #474
Add one extra page to avoid overlapping with next page if its rounded…
478: Run shellcheck on all tests r=Mic92 a=brenoguim



Co-authored-by: Breno Rodrigues Guimaraes <brenorg@gmail.com>
This prevents the test to fail when when readelf is localized.
@fengjixuchui fengjixuchui merged commit 2e22e08 into fengjixuchui:master Mar 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants