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 new and free mismatch in stale response plugin #11362

Merged
merged 1 commit into from
May 22, 2024

Conversation

bryancall
Copy link
Contributor

==408169==ERROR: AddressSanitizer: alloc-dealloc-mismatch (operator new vs free) on 0x50d000016830
    #0 0x7f45340f5558 in free.part.0 (/lib64/libasan.so.8+0xf5558) (BuildId: 6ac482fdae6aad7cf603c8e6ab3042fb9d4725af)
    #1 0x91c70f in ats_free(void*) /home/bcall/dev/apache/trafficserver/src/tscore/ink_memory.cc:127
    #2 0x7f4533efe30f in TSfree(void*) /home/bcall/dev/apache/trafficserver/src/api/InkAPI.cc:612
    #3 0x7f4514d537b7 in free_state_info /home/bcall/dev/apache/trafficserver/plugins/experimental/stale_response/stale_response.cc:198
    #4 0x7f4514d5ac36 in transaction_handler /home/bcall/dev/apache/trafficserver/plugins/experimental/stale_response/stale_response.cc:815
    #5 0x11e7d31 in INKContInternal::handle_event(int, void*) /home/bcall/dev/apache/trafficserver/src/api/InkContInternal.cc:153
    #6 0x8aa7c6 in Continuation::handleEvent(int, void*) /home/bcall/dev/apache/trafficserver/include/iocore/eventsystem/Continuation.h:228


0x50d000016830 is located 0 bytes inside of 144-byte region [0x50d000016830,0x50d0000168c0)
allocated by thread T11 here:
    #0 0x7f45340f73a8 in operator new(unsigned long) (/lib64/libasan.so.8+0xf73a8) (BuildId: 6ac482fdae6aad7cf603c8e6ab3042fb9d4725af)
    #1 0x7f4514d53365 in create_state_info /home/bcall/dev/apache/trafficserver/plugins/experimental/stale_response/stale_response.cc:141
    #2 0x7f4514d5c032 in read_request_header_handler /home/bcall/dev/apache/trafficserver/plugins/experimental/stale_response/stale_response.cc:979
    #3 0x7f4514d5c6f3 in global_request_header_hook /home/bcall/dev/apache/trafficserver/plugins/experimental/stale_response/stale_response.cc:1023
    #4 0x11e7d31 in INKContInternal::handle_event(int, void*) /home/bcall/dev/apache/trafficserver/src/api/InkContInternal.cc:153
    #5 0x8aa7c6 in Continuation::handleEvent(int, void*) /home/bcall/dev/apache/trafficserver/include/iocore/eventsystem/Continuation.h:228
    #6 0x11eaa29 in APIHook::invoke(int, void*) const /home/bcall/dev/apache/trafficserver/src/api/APIHook.cc:60

@bryancall bryancall added the ASan Address Sanitizer label May 17, 2024
@bryancall bryancall added this to the 10.1.0 milestone May 17, 2024
@bryancall bryancall self-assigned this May 17, 2024
@JosiahWI
Copy link
Collaborator

The AuTest ja3_fingerprint failed.

@JosiahWI
Copy link
Collaborator

The AuTest proxy_serve_stale_dns_fail failed.

@JosiahWI
Copy link
Collaborator

The AuTest slice_prefetch failed.

@bryancall bryancall merged commit 2bcf1d6 into apache:master May 22, 2024
15 checks passed
@cmcfarlen cmcfarlen modified the milestones: 10.1.0, 10.0.0 May 24, 2024
@cmcfarlen
Copy link
Contributor

Cherry-picked to v10.0.x

cmcfarlen pushed a commit that referenced this pull request May 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ASan Address Sanitizer
Projects
Status: picked-10.0.0
Development

Successfully merging this pull request may close these issues.

None yet

4 participants