Don't allow nullptr in texels array params (unless using a transfer buffer).

src/gpu/GrTextureProvider.cpp

 /*
  * Copyright 2015 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "GrTextureProvider.h"
 #include "GrTexturePriv.h"
 #include "GrResourceCache.h"
@@ skipping 19 matching lines @@
     {
 }
 
 GrTexture* GrTextureProvider::createMipMappedTexture(const GrSurfaceDesc& desc, SkBudgeted budgeted,
                                                      const GrMipLevel* texels, int mipLevelCount) {
     ASSERT_SINGLE_OWNER
 
     if (this->isAbandoned()) {
         return nullptr;
     }
+    if (mipLevelCount && !texels) {
+        return nullptr;
+    }
+    for (int i = 0; i < mipLevelCount; ++i) {
+        if (!texels[i].fPixels) {
+            return nullptr;
+        }
+    }
+
     if ((desc.fFlags & kRenderTarget_GrSurfaceFlag) &&
         !fGpu->caps()->isConfigRenderable(desc.fConfig, desc.fSampleCnt > 0)) {
         return nullptr;
     }
     if (!GrPixelConfigIsCompressed(desc.fConfig) &&
         !desc.fTextureStorageAllocator.fAllocateTextureStorage) {
         if (mipLevelCount < 2) {
             const GrMipLevel& baseMipLevel = texels[0];
             static const uint32_t kFlags = kExact_ScratchTextureFlag |
                                            kNoCreate_ScratchTextureFlag;
             if (GrTexture* texture = this->refScratchTexture(desc, kFlags)) {
-                if (texture->writePixels(0, 0, desc.fWidth, desc.fHeight, desc.fConfig,
-                                         baseMipLevel.fPixels, baseMipLevel.fRowBytes)) {
+                if (!texels || texture->writePixels(0, 0, desc.fWidth, desc.fHeight, desc.fConfig,
+                                                    baseMipLevel.fPixels, baseMipLevel.fRowBytes)) {
                     if (SkBudgeted::kNo == budgeted) {
                         texture->resourcePriv().makeUnbudgeted();
                     }
                     return texture;
                 }
                 texture->unref();
             }
         }
     }
 
     SkTArray<GrMipLevel> texelsShallowCopy(mipLevelCount);
     for (int i = 0; i < mipLevelCount; ++i) {
         texelsShallowCopy.push_back(texels[i]);
     }
     return fGpu->createTexture(desc, budgeted, texelsShallowCopy);
 }
 
 GrTexture* GrTextureProvider::createTexture(const GrSurfaceDesc& desc, SkBudgeted budgeted,
                                             const void* srcData, size_t rowBytes) {
-    const int mipLevelCount = 1;
-    GrMipLevel texels[mipLevelCount];
-    texels[0].fPixels = srcData;
-    texels[0].fRowBytes = rowBytes;
-
-    return this->createMipMappedTexture(desc, budgeted, texels, mipLevelCount);
+    if (srcData) {
+        GrMipLevel texels;
+        texels.fPixels = srcData;
+        texels.fRowBytes = rowBytes;
+        return this->createMipMappedTexture(desc, budgeted, &texels, 1);
+    }
+    return this->createMipMappedTexture(desc, budgeted, nullptr, 0);
 }
 
 GrTexture* GrTextureProvider::createApproxTexture(const GrSurfaceDesc& desc) {
     ASSERT_SINGLE_OWNER
     return this->internalCreateApproxTexture(desc, 0);
 }
 
 GrTexture* GrTextureProvider::internalCreateApproxTexture(const GrSurfaceDesc& desc,
                                                           uint32_t scratchFlags) {
     ASSERT_SINGLE_OWNER
@@ skipping 91 matching lines @@
 GrTexture* GrTextureProvider::findAndRefTextureByUniqueKey(const GrUniqueKey& key) {
     ASSERT_SINGLE_OWNER
     GrGpuResource* resource = this->findAndRefResourceByUniqueKey(key);
     if (resource) {
         GrTexture* texture = static_cast<GrSurface*>(resource)->asTexture();
         SkASSERT(texture);
         return texture;
     }
     return NULL;
 }