Skip to content

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

Merged
fengjixuchui merged 64 commits intofengjixuchui:masterfrom
NixOS:master
Mar 27, 2023
Merged

Merge pull request NixOS#488 from cedric84/patch-1#11
fengjixuchui merged 64 commits intofengjixuchui:masterfrom
NixOS:master

Conversation

@fengjixuchui
Copy link
Copy Markdown
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] <[email protected]>
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 <[email protected]>
456: Add options to print, clear and set executable stack state r=Mic92 a=cgzones



Co-authored-by: Christian Göttsche <[email protected]>
463: Apply fixes from shellcheck r=Mic92 a=Mic92



Co-authored-by: Jörg Thalheim <[email protected]>
Co-authored-by: Jörg Thalheim <[email protected]>
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 <[email protected]>
Co-authored-by: Jörg Thalheim <[email protected]>
Co-authored-by: Jörg Thalheim <[email protected]>
Co-authored-by: Jörg Thalheim <[email protected]>
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 <[email protected]>
Co-authored-by: Breno Rodrigues Guimarães <[email protected]>
Co-authored-by: Jörg Thalheim <[email protected]>
466: Adjust roundUp for 0 as input r=Mic92 a=cgzones



Co-authored-by: Christian Göttsche <[email protected]>
460: Avoid overlapping program header table with section header table #457 r=Mic92 a=brenoguim



Co-authored-by: Breno Rodrigues Guimaraes <[email protected]>
Co-authored-by: Breno Rodrigues Guimarães <[email protected]>
Co-authored-by: Jörg Thalheim <[email protected]>
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] <[email protected]>
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 <[email protected]>
Co-authored-by: Jörg Thalheim <[email protected]>
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 <[email protected]>
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