[openssl/openssl] 75ecda: CI: add Clang 15

Sam James noreply at github.com
Fri Oct 21 14:01:53 UTC 2022


  Branch: refs/heads/master
  Home:   https://github.com/openssl/openssl
  Commit: 75ecda930e0a961f9605ce090af64d95c98ed161
      https://github.com/openssl/openssl/commit/75ecda930e0a961f9605ce090af64d95c98ed161
  Author: Sam James <sam at gentoo.org>
  Date:   2022-10-21 (Fri, 21 Oct 2022)

  Changed paths:
    M .github/workflows/compiler-zoo.yml

  Log Message:
  -----------
  CI: add Clang 15

We have to use the PPA provided by LLVM because Clang 15 isn't
officially part of Ubuntu 22.04 (or any other Ubuntu release yet),
see https://apt.llvm.org/ for details.

Signed-off-by: Sam James <sam at gentoo.org>

Reviewed-by: Paul Dale <pauli at openssl.org>
Reviewed-by: Hugo Landau <hlandau at openssl.org>
Reviewed-by: Tomas Mraz <tomas at openssl.org>
(Merged from https://github.com/openssl/openssl/pull/19450)


  Commit: 71bc497dc321adeb08e7541556dea019c81c9a87
      https://github.com/openssl/openssl/commit/71bc497dc321adeb08e7541556dea019c81c9a87
  Author: Sam James <sam at gentoo.org>
  Date:   2022-10-21 (Fri, 21 Oct 2022)

  Changed paths:
    M crypto/pem/pem_lib.c

  Log Message:
  -----------
  pem: fix -Wunused-but-set-variable

The loop never uses the value of 'line'.

Fixes this error with Clang 15:
```
crypto/pem/pem_lib.c:821:14: error: variable 'line' set but not used [-Werror,-Wunused-but-set-variable]
    int len, line, ret = 0, end = 0, prev_partial_line_read = 0, partial_line_read = 0;
             ^
1 error generated.
```

Signed-off-by: Sam James <sam at gentoo.org>

Reviewed-by: Paul Dale <pauli at openssl.org>
Reviewed-by: Hugo Landau <hlandau at openssl.org>
Reviewed-by: Tomas Mraz <tomas at openssl.org>
(Merged from https://github.com/openssl/openssl/pull/19450)


  Commit: f9e8e2c0ab73409862bb78a9285c1b72e0511750
      https://github.com/openssl/openssl/commit/f9e8e2c0ab73409862bb78a9285c1b72e0511750
  Author: Sam James <sam at gentoo.org>
  Date:   2022-10-21 (Fri, 21 Oct 2022)

  Changed paths:
    M crypto/txt_db/txt_db.c

  Log Message:
  -----------
  txt_db: fix -Wunused-but-set-variable

The loop never uses the value of 'ln'.

Fixes this error with Clang 15:
```
crypto/txt_db/txt_db.c:24:10: error: variable 'ln' set but not used [-Werror,-Wunused-but-set-variable]
    long ln = 0;
         ^
1 error generated.
```

Signed-off-by: Sam James <sam at gentoo.org>

Reviewed-by: Paul Dale <pauli at openssl.org>
Reviewed-by: Hugo Landau <hlandau at openssl.org>
Reviewed-by: Tomas Mraz <tomas at openssl.org>
(Merged from https://github.com/openssl/openssl/pull/19450)


  Commit: c71318668571b3680fe10035a1a350ff46e459af
      https://github.com/openssl/openssl/commit/c71318668571b3680fe10035a1a350ff46e459af
  Author: Sam James <sam at gentoo.org>
  Date:   2022-10-21 (Fri, 21 Oct 2022)

  Changed paths:
    M crypto/x509/x_name.c

  Log Message:
  -----------
  x509: fix -Wunused-but-set-variable

The value of 'l' isn't ever actually used.

Fixes this error with Clang 15:
```
crypto/x509/x_name.c:506:9: error: variable 'l' set but not used [-Werror,-Wunused-but-set-variable]
    int l, i;
        ^
1 error generated.
```

Signed-off-by: Sam James <sam at gentoo.org>

Reviewed-by: Paul Dale <pauli at openssl.org>
Reviewed-by: Hugo Landau <hlandau at openssl.org>
Reviewed-by: Tomas Mraz <tomas at openssl.org>
(Merged from https://github.com/openssl/openssl/pull/19450)


  Commit: 6a94c5849ea7d1f08d4fcaa9a6fc0a947e19da66
      https://github.com/openssl/openssl/commit/6a94c5849ea7d1f08d4fcaa9a6fc0a947e19da66
  Author: Sam James <sam at gentoo.org>
  Date:   2022-10-21 (Fri, 21 Oct 2022)

  Changed paths:
    M test/testutil/driver.c

  Log Message:
  -----------
  test: driver: fix -Wunused-but-set-variable

The value of 'num_failed_inner' isn't ever used.

Fixes this error with Clang 15:
```
test/testutil/driver.c:341:17: error: variable 'num_failed_inner' set but not used [-Werror,-Wunused-but-set-variable]
            int num_failed_inner = 0;
                ^
1 error generated.
```

Signed-off-by: Sam James <sam at gentoo.org>

Reviewed-by: Paul Dale <pauli at openssl.org>
Reviewed-by: Hugo Landau <hlandau at openssl.org>
Reviewed-by: Tomas Mraz <tomas at openssl.org>
(Merged from https://github.com/openssl/openssl/pull/19450)


Compare: https://github.com/openssl/openssl/compare/bd19999b396d...6a94c5849ea7


More information about the openssl-commits mailing list