diff options
| author | Olivier Dugeon <olivier.dugeon@orange.com> | 2022-08-01 17:26:36 +0200 |
|---|---|---|
| committer | Olivier Dugeon <olivier.dugeon@orange.com> | 2022-08-01 17:32:56 +0200 |
| commit | 9bf9bd1a0b818f8cd1554ef07d47028247703b75 (patch) | |
| tree | ef863cb9f620eb9bd43a1224e1beedba8aeb2606 /lib/link_state.h | |
| parent | Merge pull request #11691 from mxyns/bmp-vpnv4monupd-fix (diff) | |
| download | frr-9bf9bd1a0b818f8cd1554ef07d47028247703b75.tar.xz frr-9bf9bd1a0b818f8cd1554ef07d47028247703b75.zip | |
lib: Correct valgrind errors
In CSPF topo test, valgrind detects uninitialized bytes when exporting TE
Opaque information through ZEBRA. This is due to C pragma compilation directive
__attribute__(aligned(8)) in struct ls_node_id in link_state.h. Valgrind
consideris that struct ls_node_id nid = {} doesn't initialized the padding
bytes introduced by gcc.
This patch simply removes the C pragma compilation directive and also takes
opportunity to remove the transmission of remote node id for vertices and
subnets which is not known. Indeed, remote node id is only pertinent for
edges.
Signed-off-by: Olivier Dugeon <olivier.dugeon@orange.com>
Diffstat (limited to '')
| -rw-r--r-- | lib/link_state.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/link_state.h b/lib/link_state.h index f46a2068a..ed315452d 100644 --- a/lib/link_state.h +++ b/lib/link_state.h @@ -91,7 +91,7 @@ struct ls_node_id { uint8_t level; /* ISIS Level */ uint8_t padding; } iso; - } id __attribute__((aligned(8))); + } id; }; /** |