diff options
author | android-build-prod (mdb) <android-build-team-robot@google.com> | 2019-10-08 00:29:46 +0000 |
---|---|---|
committer | android-build-prod (mdb) <android-build-team-robot@google.com> | 2019-10-08 00:29:46 +0000 |
commit | 0b78c77856b994f13e169779ab89b4a27786036a (patch) | |
tree | 994917bfe4c52a7ef203b8e4269402670aeef5fc | |
parent | 36d99be5a35c744a4f56dfcab297b17e7a3e29ac (diff) | |
parent | 607711c60de5a025d041b44476ef4145bec456c3 (diff) | |
download | opengl-transport-platform-tools-29.0.5.tar.gz |
Snap for 5925869 from 607711c60de5a025d041b44476ef4145bec456c3 to sdk-releaseplatform-tools-29.0.5
Change-Id: I408d6c829b49362d31f1985a807d9b779310a963
-rw-r--r-- | host/libs/virglrenderer/AVDVirglRenderer.cpp | 8 | ||||
-rw-r--r-- | host/libs/virglrenderer/include/hardware/gralloc.h | 5 |
2 files changed, 12 insertions, 1 deletions
diff --git a/host/libs/virglrenderer/AVDVirglRenderer.cpp b/host/libs/virglrenderer/AVDVirglRenderer.cpp index 95e725600..973e14442 100644 --- a/host/libs/virglrenderer/AVDVirglRenderer.cpp +++ b/host/libs/virglrenderer/AVDVirglRenderer.cpp @@ -340,6 +340,14 @@ int virgl_renderer_init(void* cookie, int flags, virgl_renderer_callbacks* cb) { g_dpy = EGL_NO_DISPLAY; return ENOENT; } + + // Our static analyzer sees the `new`ing of `config` below without any sort + // of attempt to free it, and warns about it. Normally, it would catch that + // we're pushing it into a vector in the constructor, but it hits an + // internal evaluation limit when trying to evaluate the loop inside of the + // ctor. So, it never gets to see that we escape our newly-allocated + // `config` instance. Silence the warning, since it's incorrect. + // NOLINTNEXTLINE(clang-analyzer-cplusplus.NewDeleteLeaks) for (EGLint c = 0; c < nConfigs; c++) { EGLint configId; if (!s_egl.eglGetConfigAttrib(g_dpy, configs[c], EGL_CONFIG_ID, &configId)) { diff --git a/host/libs/virglrenderer/include/hardware/gralloc.h b/host/libs/virglrenderer/include/hardware/gralloc.h index 8446ba4ea..3847164ed 100644 --- a/host/libs/virglrenderer/include/hardware/gralloc.h +++ b/host/libs/virglrenderer/include/hardware/gralloc.h @@ -42,5 +42,8 @@ struct gralloc_module_t { int (*unlockAsync)(gralloc_module_t const*, buffer_handle_t, int*); int (*lockAsync_ycbcr)(gralloc_module_t const*, buffer_handle_t, int, int, int, int, int, android_ycbcr*, int); - void* reserved_proc[3]; + int32_t (*getTransportSize)(gralloc_module_t const*, buffer_handle_t, uint32_t, uint32_t); + int32_t (*validateBufferSize)(gralloc_module_t const*, buffer_handle_t, uint32_t, uint32_t, + int32_t, int, uint32_t); + void* reserved_proc[1]; }; |