Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Chopps/mgmt dbg improve #14936

Merged

Conversation

choppsv1
Copy link
Contributor

@choppsv1 choppsv1 commented Dec 4, 2023

depends on #14934

@frrbot frrbot bot added libfrr mgmt FRR Management Infra tests Topotests, make check, etc labels Dec 4, 2023
@choppsv1 choppsv1 force-pushed the chopps/mgmt-dbg-improve branch from fc127e1 to 38b44ae Compare December 5, 2023 18:39
@github-actions github-actions bot added size/L and removed size/XL labels Dec 5, 2023
@donaldsharp donaldsharp merged commit 2df4683 into FRRouting:master Dec 7, 2023
77 checks passed
@louis-6wind
Copy link
Contributor

I get this issue since this patch:

# /usr/lib/frr/staticd
=================================================================
==303023==ERROR: AddressSanitizer: odr-violation (0x55e881e856a0):
  [1] size=8 'mgmt_be_client' staticd/static_main.c:56:24
  [2] size=8 'mgmt_be_client' lib/mgmt_be_client.c:119:24
These globals were registered at these points:
  [1]:
    #0 0x7fb7d2a36f40 in __asan_register_globals ../../../../src/libsanitizer/asan/asan_globals.cpp:341
    #1 0x55e881e2df4d in _sub_I_00099_1 (/usr/lib/frr/staticd+0x15f4d)
    #2 0x55e881e61a34 in __libc_csu_init (/usr/lib/frr/staticd+0x49a34)

  [2]:
    #0 0x7fb7d2a36f40 in __asan_register_globals ../../../../src/libsanitizer/asan/asan_globals.cpp:341
    #1 0x7fb7d264692f in _sub_I_00099_1 (/lib/libfrr.so.0+0x24692f)
    #2 0x7fb7d35e9fe1 in call_init elf/dl-init.c:72

==303023==HINT: if you don't care about these errors you may set ASAN_OPTIONS=detect_odr_violation=0
SUMMARY: AddressSanitizer: odr-violation: global 'mgmt_be_client' at staticd/static_main.c:56:24
==303023==ABORTING

Compilation on debian 11 with
./configure CFLAGS='-g -O0' --prefix=/usr --localstatedir=/var/run/frr --sbindir=/usr/lib/frr --sysconfdir=/etc/frr --enable-vtysh --enable-pimd --enable-pim6d --enable-sharpd --enable-staticd --enable-multipath=64 --enable-user=root --enable-group=root --enable-vty-group=root --enable-snmp=agentx --disable-capabilities --enable-address-sanitizer --with-pkg-extra-version=-my-manual-build --enable-fabricd --enable-werror

@choppsv1 choppsv1 deleted the chopps/mgmt-dbg-improve branch January 9, 2024 21:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
libfrr master mgmt FRR Management Infra size/L tests Topotests, make check, etc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants