!94 fixed code check bugs

Merge pull request !94 from 韦德立/master

external/android-emugl/shared/OpenglCodecCommon/ChecksumCalculatorThreadInfo.cpp

@@ -98,7 +98,7 @@ bool ChecksumCalculatorThreadInfo::writeChecksum(const void* buf,
 
 bool ChecksumCalculatorThreadInfo::validate(const void* buf,
                                             size_t bufLen,
-                                            void* checksum,
+                                            const void* checksum,
                                             size_t checksumLen) {
     bool ret = false;
     if(getChecksumCalculatorThreadInfo() != nullptr) {

external/android-emugl/shared/OpenglCodecCommon/ChecksumCalculatorThreadInfo.h

@@ -43,7 +43,7 @@ public:
 
     static bool validate(const void* buf,
                          size_t bufLen,
-                         void* checksum,
+                         const void* checksum,
                          size_t checksumLen);
     static void validOrDie(const void* buf,
                            size_t bufLen,

src/anbox/android/ip_config_builder.h

@@ -39,6 +39,7 @@ class IpConfigBuilder {
   };
 
   IpConfigBuilder() = default;
+  ~IpConfigBuilder(){}
 
   std::size_t write(common::BinaryWriter &writer);
 

src/anbox/graphics/emugl/ColorBuffer.cpp

@@ -295,6 +295,9 @@ bool ColorBuffer::blitFromCurrentReadBuffer() {
     s_gles2.glViewport(0, 0, m_width, m_height);
 
     // render m_blitTex
+    if (!m_helper->getTextureDraw()) {
+        return false;
+    }
     m_helper->getTextureDraw()->draw(m_blitTex);
 
     // Restore previous viewport.

src/anbox/graphics/emugl/Renderer.cpp

@@ -431,6 +431,9 @@ HandleType Renderer::createRenderContext(int p_config, HandleType p_share,
     ret = genHandle();
     m_contexts[ret] = rctx;
     RenderThreadInfo *tinfo = RenderThreadInfo::get();
+    if (!tinfo) {
+      return -1;
+    }
     int tid = tinfo->m_tid;
     // The new emulator manages render contexts per guest process.
     // Fall back to per-thread management if the system image does not
@@ -511,6 +514,9 @@ void Renderer::DestroyRenderContext(HandleType p_context) {
 
   m_contexts.erase(p_context);
   RenderThreadInfo *tinfo = RenderThreadInfo::get();
+  if (!tinfo) {
+    return;
+  }
     int tid = tinfo->m_tid;
     // The new emulator manages render contexts per guest process.
     // Fall back to per-thread management if the system image does not