Update ci and ABI xml files to validate function parameters

libabigail is currenly only validating symbol presence and version
information in ci.  We should also be validating function parameters,
types, etc.  To do this we need to build the library with -g so the
dwarf information is available for libabigail to interrogate

while we're at it, also add a script to re-generate the xml that abidiff
uses for comparison during ci runs, to make updates easier

Fixes #22712

Reviewed-by: Hugo Landau <hlandau@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/22713)
This commit is contained in:
Neil Horman 2023-11-13 04:19:18 -05:00 committed by Tomas Mraz
parent 46376fcf4b
commit dcfd8cfd4a
4 changed files with 76837 additions and 5 deletions

View File

@ -279,7 +279,7 @@ jobs:
- name: install extra config support
run: sudo apt-get -y install libsctp-dev abigail-tools libzstd-dev zstd
- name: config
run: ./config --banner=Configured --strict-warnings enable-ktls enable-fips enable-egd enable-ec_nistp_64_gcc_128 enable-md2 enable-rc5 enable-sctp enable-ssl3 enable-ssl3-method enable-trace enable-zlib enable-zstd && perl configdata.pm --dump
run: ./config --banner=Configured -g --strict-warnings enable-ktls enable-fips enable-egd enable-ec_nistp_64_gcc_128 enable-md2 enable-rc5 enable-sctp enable-ssl3 enable-ssl3-method enable-trace enable-zlib enable-zstd && perl configdata.pm --dump
- name: make
run: make -s -j4
- name: get cpu info

File diff suppressed because it is too large Load Diff

File diff suppressed because it is too large Load Diff

5
util/update_abi_check.sh Executable file
View File

@ -0,0 +1,5 @@
#!/bin/sh
#
abidw --out-file ./.github/workflows/libcrypto-abi.xml libcrypto.so
abidw --out-file ./.github/workflows/libssl-abi.xml libssl.so