Skip to content

Conversation

sjaeckel
Copy link
Member

No description provided.

@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
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 <[email protected]>
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 <[email protected]>
c.f. [0] for details.

[0] actions/runner-images#8397

Signed-off-by: Steffen Jaeckel <[email protected]>
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 <[email protected]>
@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