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

compel: correct the syscall number of bind on ARM64 #2301

Merged
merged 1 commit into from
Nov 28, 2023

Conversation

SallyKAN
Copy link
Contributor

In the compel/arch/arm/plugins/std/syscalls/syscall.def, the syscall number of bind on ARM64 should be 200 instead of 235
Referring to this

In the compel/arch/arm/plugins/std/syscalls/syscall.def, the syscall number of bind on ARM64 should be 200 instead of 235

Signed-off-by: Sally Kang <[email protected]>
@SallyKAN SallyKAN changed the title comple: correct the syscall number of bind on ARM64 compel: correct the syscall number of bind on ARM64 Nov 28, 2023
@adrianreber
Copy link
Member

Looks good. Thanks.

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (c474816) 70.55% compared to head (b3c6833) 70.59%.

❗ Current head b3c6833 differs from pull request most recent head 3dfb675. Consider uploading reports for the commit 3dfb675 to get more accurate results

Additional details and impacted files
@@             Coverage Diff              @@
##           criu-dev    #2301      +/-   ##
============================================
+ Coverage     70.55%   70.59%   +0.03%     
============================================
  Files           132      134       +2     
  Lines         33508    33306     -202     
============================================
- Hits          23642    23512     -130     
+ Misses         9866     9794      -72     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@avagin avagin merged commit d88dcef into checkpoint-restore:criu-dev Nov 28, 2023
33 of 37 checks passed
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.

4 participants