mapbox / mapbox-gl-native

Interactive, thoroughly customizable maps in native Android, iOS, macOS, Node.js, and Qt applications, powered by vector tiles and OpenGL
https://mapbox.com/mobile
Other
4.36k stars 1.33k forks source link

GeoJSONTile::querySourceFeatures #9840

Closed tobrun closed 6 years ago

tobrun commented 7 years ago

Follow up from downstream issue in https://github.com/mapbox/mapbox-plugins-android/issues/77. In the code from the location layer plugin we are queering the map every time the location changes.

Symbolicated stacktrace:

MapboxGLAndroidSDK/build/intermediates/cmake/debug/obj/armeabi-v7a/ 
********** Crash dump: **********
Build fingerprint: 'samsung/j3xltejt/j3xlte:5.1.1/LMY47V/J320FXXU0AQG2:user/release-keys'
pid: 28966, tid: 28966, name: pboxsdk.testapp  >>> com.mapbox.mapboxsdk.testapp <<<
signal 11 (SIGSEGV), code 1 (SEGV_MAPERR), fault addr 0x0
Stack frame #00 pc 005a52d0  /data/app/com.mapbox.mapboxsdk.testapp-1/lib/arm/libmapbox-gl.so: Routine mbgl::GeoJSONTile::querySourceFeatures(std::__ndk1::vector<mapbox::geometry::feature<double>, std::__ndk1::allocator<mapbox::geometry::feature<double> > >&, mbgl::SourceQueryOptions const&) at /home/tvn/Mapbox/mapbox-gl-native/src/mbgl/tile/geojson_tile.cpp:121
Stack frame #01 pc 005c4e31  /data/app/com.mapbox.mapboxsdk.testapp-1/lib/arm/libmapbox-gl.so: Routine mbgl::TilePyramid::querySourceFeatures(mbgl::SourceQueryOptions const&) const at /home/tvn/Mapbox/mapbox-gl-native/src/mbgl/renderer/tile_pyramid.cpp:263
Stack frame #02 pc 0058a165  /data/app/com.mapbox.mapboxsdk.testapp-1/lib/arm/libmapbox-gl.so: Routine mbgl::RenderGeoJSONSource::querySourceFeatures(mbgl::SourceQueryOptions const&) const at /home/tvn/Mapbox/mapbox-gl-native/src/mbgl/renderer/sources/render_geojson_source.cpp:84
Stack frame #03 pc 001edc8d  /data/app/com.mapbox.mapboxsdk.testapp-1/lib/arm/libmapbox-gl.so: Routine mbgl::Renderer::Impl::querySourceFeatures(std::__ndk1::basic_string<char, std::__ndk1::char_traits<char>, std::__ndk1::allocator<char> > const&, mbgl::SourceQueryOptions const&) const at /home/tvn/Mapbox/mapbox-gl-native/src/mbgl/renderer/renderer_impl.cpp:339
Stack frame #04 pc 001e8983  /data/app/com.mapbox.mapboxsdk.testapp-1/lib/arm/libmapbox-gl.so: Routine mbgl::Renderer::querySourceFeatures(std::__ndk1::basic_string<char, std::__ndk1::char_traits<char>, std::__ndk1::allocator<char> > const&, mbgl::SourceQueryOptions const&) const at /home/tvn/Mapbox/mapbox-gl-native/src/mbgl/renderer/renderer.cpp:67
Stack frame #05 pc 002f7987  /data/app/com.mapbox.mapboxsdk.testapp-1/lib/arm/libmapbox-gl.so: Routine mbgl::android::AndroidRendererFrontend::querySourceFeatures(std::__ndk1::basic_string<char, std::__ndk1::char_traits<char>, std::__ndk1::allocator<char> > const&, mbgl::SourceQueryOptions const&) const at /home/tvn/Mapbox/mapbox-gl-native/platform/android/src/android_renderer_frontend.cpp:55
Stack frame #06 pc 001d1305  /data/app/com.mapbox.mapboxsdk.testapp-1/lib/arm/libmapbox-gl.so: Routine mbgl::android::GeoJSONSource::querySourceFeatures(_JNIEnv&, jni::Array<jni::Object<jni::ObjectTag>, void>) at /home/tvn/Mapbox/mapbox-gl-native/platform/android/src/style/sources/geojson_source.cpp:114
Stack frame #07 pc 001d62eb  /data/app/com.mapbox.mapboxsdk.testapp-1/lib/arm/libmapbox-gl.so: Routine operator() at /home/tvn/Mapbox/mapbox-gl-native/platform/android/MapboxGLAndroidSDK/.externalNativeBuild/cmake/debug/armeabi-v7a/../../../../../../../mason_packages/headers/jni.hpp/3.0.0/include/jni/native_method.hpp:245
Stack frame #08 pc 001d6245  /data/app/com.mapbox.mapboxsdk.testapp-1/lib/arm/libmapbox-gl.so: Routine operator() at /home/tvn/Mapbox/mapbox-gl-native/platform/android/MapboxGLAndroidSDK/.externalNativeBuild/cmake/debug/armeabi-v7a/../../../../../../../mason_packages/headers/jni.hpp/3.0.0/include/jni/native_method.hpp:112
Stack frame #09 pc 001d61e7  /data/app/com.mapbox.mapboxsdk.testapp-1/lib/arm/libmapbox-gl.so: Routine __invoke at /home/tvn/Mapbox/mapbox-gl-native/platform/android/MapboxGLAndroidSDK/.externalNativeBuild/cmake/debug/armeabi-v7a/../../../../../../../mason_packages/headers/jni.hpp/3.0.0/include/jni/native_method.hpp:110
Stack frame #10 pc 001d639b  /data/app/com.mapbox.mapboxsdk.testapp-1/lib/arm/libmapbox-gl.so: Routine operator()<jni::jobject *, jni::jarray<jni::jobject> *> at /home/tvn/Mapbox/mapbox-gl-native/platform/android/MapboxGLAndroidSDK/.externalNativeBuild/cmake/debug/armeabi-v7a/../../../../../../../mason_packages/headers/jni.hpp/3.0.0/include/jni/native_method.hpp:59
Stack frame #11 pc 001d634b  /data/app/com.mapbox.mapboxsdk.testapp-1/lib/arm/libmapbox-gl.so: Routine __invoke<jni::jobject *, jni::jarray<jni::jobject> *> at /home/tvn/Mapbox/mapbox-gl-native/platform/android/MapboxGLAndroidSDK/.externalNativeBuild/cmake/debug/armeabi-v7a/../../../../../../../mason_packages/headers/jni.hpp/3.0.0/include/jni/native_method.hpp:55
Stack frame #12 pc 00187453  /data/dalvik-cache/arm/data@app@com.mapbox.mapboxsdk.testapp-1@base.apk@classes.dex
jfirebaugh commented 6 years ago

Fixed in #10287.