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

Update CI #587

Merged
merged 7 commits into from
Feb 24, 2025
Merged

Update CI #587

merged 7 commits into from
Feb 24, 2025

Conversation

sjaeckel
Copy link
Member

No description provided.

Signed-off-by: Steffen Jaeckel <s@jaeckel.eu>
Signed-off-by: Steffen Jaeckel <s@jaeckel.eu>
@sjaeckel sjaeckel changed the title Make it possible to deploy for Ubuntu 24.04 Update CI Feb 24, 2025
@sjaeckel sjaeckel requested a review from levitte February 24, 2025 12:17
Signed-off-by: Steffen Jaeckel <s@jaeckel.eu>
Warning was:

```
In function ‘mp_clear’,
    inlined from ‘mp_gcd.part.0’ at mp_gcd.c:89:4:
mp_clear.c:10:9: warning: ‘v.dp’ may be used uninitialized [-Wmaybe-uninitialized]
   10 |    if (a->dp != NULL) {
      |         ^
```

Fixes: aab1b3d ("split label & statement")

Signed-off-by: Steffen Jaeckel <s@jaeckel.eu>
The warning was:

```
s_mp_exptmod_fast.c: In function ‘s_mp_exptmod_fast’:
s_mp_exptmod_fast.c:188:21: warning: ‘mp’ may be used uninitialized in this function [-Wmaybe-uninitialized]
  188 |          if ((err = redux(&res, P, mp)) != MP_OKAY)               goto LBL_RES;
      |                     ^
s_mp_exptmod_fast.c:25:18: note: ‘mp’ was declared here
   25 |    mp_digit buf, mp;
      |                  ^
```

Signed-off-by: Steffen Jaeckel <s@jaeckel.eu>
c.f. [0] for details.

[0] actions/runner-images#8397

Signed-off-by: Steffen Jaeckel <s@jaeckel.eu>
It terminates with the following error message:

```
==2028==ERROR: UndefinedBehaviorSanitizer failed to allocate 0x0 (0) bytes of SetAlternateSignalStack (error code: 22)
==2028==Process memory map follows:
[memory map]
==2028==End of process memory map.
==2028==Sanitizer CHECK failed: /build/llvm-toolchain-11-mnvtwk/llvm-toolchain-11-11.1.0/compiler-rt/lib/sanitizer_common/sanitizer_common.cpp:54 ((0 && "unable to mmap")) != (0) (0, 0)
```

Signed-off-by: Steffen Jaeckel <s@jaeckel.eu>
@sjaeckel sjaeckel merged commit b996311 into develop Feb 24, 2025
80 checks passed
@sjaeckel sjaeckel deleted the update-ci branch February 24, 2025 16:12
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.

2 participants