diff options
| author | Ilya Maximets <i.maximets@ovn.org> | 2022-09-15 14:17:30 +0200 |
|---|---|---|
| committer | Ilya Maximets <i.maximets@ovn.org> | 2022-09-16 12:53:41 +0200 |
| commit | 2e74c44756137d624d8f0e4418e91275849d9622 (patch) | |
| tree | 41d550682c39eb6581e1dbe2759447363249a68b /tests | |
| parent | a93d0b74dd0a4519e04441144756ffb056b4e8c5 (diff) | |
| download | openvswitch-2e74c44756137d624d8f0e4418e91275849d9622.tar.gz | |
test-list: Fix false-positive build failure with GCC 12.
GCC 12.2.1 on Fedora 36 generates the following false-positive
warning that is treated as error with -Werror:
tests/test-list.c: In function 'test_list_construction':
tests/test-list.c:110:9: error: 'values' may be used uninitialized
110 | check_list(&list, values, n);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
For some reason it fails to recognize that array will not
be used if 'n' equals zero.
Fix that by just initializing arrays in full before using,
since it's just a test code.
Acked-by: Eelco Chaudron <echaudro@redhat.com>
Signed-off-by: Ilya Maximets <i.maximets@ovn.org>
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/test-list.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/tests/test-list.c b/tests/test-list.c index 2c6c44448..ac82f2048 100644 --- a/tests/test-list.c +++ b/tests/test-list.c @@ -106,6 +106,8 @@ test_list_construction(void) int values[MAX_ELEMS]; struct ovs_list list; + memset(elements, 0, sizeof elements); + memset(values, 0, sizeof values); make_list(&list, elements, values, n); check_list(&list, values, n); } |