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

ASAN crash in master: SEGV zebra/zebra_pw.c:137 in zebra_pw_enabled #17041

Closed
2 tasks done
mwinter-osr opened this issue Oct 9, 2024 · 0 comments · Fixed by #17042
Closed
2 tasks done

ASAN crash in master: SEGV zebra/zebra_pw.c:137 in zebra_pw_enabled #17041

mwinter-osr opened this issue Oct 9, 2024 · 0 comments · Fixed by #17042
Labels
triage Needs further investigation

Comments

@mwinter-osr
Copy link
Member

Description

Since sha a4fb596 we have a crash in the ASAN testing of FRR master in ldp_snmp: test_ldp_snmp_topo1 test

# cat ldp_snmp.test_ldp_snmp_topo1/r1.asan.zebra.31837
=================================================================
==31837==ERROR: AddressSanitizer: SEGV on unknown address 0x0000000000ac (pc 0x56046c00d496 bp 0x7ffd69950950 sp 0x7ffd69950950 T0)
==31837==The signal is caused by a READ memory access.
==31837==Hint: address points to the zero page.
    #0 0x56046c00d496 in zebra_pw_enabled zebra/zebra_pw.c:137
    #1 0x56046c011426 in zebra_pw_handle_dplane_results zebra/zebra_pw.c:214
    #2 0x56046c01981d in rib_process_dplane_results zebra/zebra_rib.c:4959
    #3 0x7fc5cf2da301 in event_call lib/event.c:1996
    #4 0x7fc5cf20dcf6 in frr_run lib/libfrr.c:1238
    #5 0x56046bf2f3b3 in main zebra/main.c:526
    #6 0x7fc5ced0c249 in __libc_start_call_main ../sysdeps/nptl/libc_start_call_main.h:58
    #7 0x7fc5ced0c304 in __libc_start_main_impl ../csu/libc-start.c:360
    #8 0x56046bf03790 in _start (/usr/lib/frr/zebra+0x1a4790)

AddressSanitizer can not provide additional info.
SUMMARY: AddressSanitizer: SEGV zebra/zebra_pw.c:137 in zebra_pw_enabled
==31837==ABORTING

Most likely candidate is PR #17009

Version

Master as of sha a4fb596 and newer

How to reproduce

ASAN run (AddressSanitizer Debian 12 amd64 Part 7 on CI)

Expected behavior

passing

Actual behavior

crashing

Additional context

No response

Checklist

  • I have searched the open issues for this bug.
  • I have not included sensitive information in this report.
@mwinter-osr mwinter-osr added the triage Needs further investigation label Oct 9, 2024
@Jafaral Jafaral closed this as completed in 154a89b Oct 9, 2024
mergify bot pushed a commit that referenced this issue Oct 9, 2024
Recent PR #17009 introduced a crash in pw handing
for deletion.  Let's fix that problem.

Fixes: #17041
Signed-off-by: Donald Sharp <sharpd@nvidia.com>
(cherry picked from commit 154a89b)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
triage Needs further investigation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant