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

add LSAN support #431

Open
zhihuawensc opened this issue Jun 24, 2017 · 13 comments
Open

add LSAN support #431

zhihuawensc opened this issue Jun 24, 2017 · 13 comments
Milestone

Comments

@zhihuawensc
Copy link

zhihuawensc commented Jun 24, 2017


Description

Not sure if LeakSanitizer is supported on
on pixel with userdebug build android 7.1.2

Environment Details

Platform - OSX 10.12.5
Device - Google Pixel, userdebug build NJH47B android-7.1.2_r17
NDK 15.1.4119039

I was trying to use Address Sanitizer on Anroid, I built the 7.1.2 userdebug version on Pixel.

After run ./asan_device_setup, I was able to catch simple use-after-free, stack-overflow and heap-buffer-overflow. However, ASAN seems cant detect simple memory leaks, is LSAN enabled on Android now.

@DanAlbert
Copy link
Member

I'm actually not sure. LSAN is part of ASAN iirc, but idk if that's true for Android. @eugenis or @stephenhines would know.

@eugenis
Copy link
Collaborator

eugenis commented Jun 27, 2017

Not yet, sorry. See google/sanitizers#379

@infomaniac777
Copy link

@eugenis Is there any timeline for Leak Sanitizer support in Android?

@infomaniac777
Copy link

I'm sure a lot of people are looking forward to this.

@DanAlbert
Copy link
Member

@stephenhines: is this on the roadmap for the platform/toolchain?

@stephenhines
Copy link
Collaborator

No, we have no timeline for when LSan will be available in Android.

@infomaniac777
Copy link

@stephenhines I have found LSan to be very useful in debugging memory leaks. Is there not enough demand for LSan on Android ?

@stephenhines
Copy link
Collaborator

It just isn't a priority at this time. We don't have the ability to take on every project that could be helpful.

@infomaniac777
Copy link

Ok. I understand.

@axnsan12
Copy link

Judging by google/sanitizers#379 (comment) and https://reviews.llvm.org/D85927 LSAN should now be supported on 30/31+?

Is there any update on the NDK side?

@stephenhines
Copy link
Collaborator

This is a feature request, and official support requires additional testing if we are going to advertise it in the NDK. I think that would be the biggest remaining blocker from considering this "Fixed".

@20083017
Copy link

android still not support lsan yet? hope for a quick answer~

@DanAlbert
Copy link
Member

DanAlbert commented Feb 23, 2024

The quick answers are on this page. The bug is open and it's in the "unplanned" milestone.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants