commit 2af4599b5c514933bf997d4837ddaaf24fc61cd7
author lsalzman <lsalzman@mozilla.com> Tue Jun 07 19:08:57 2016 -0700
committer Commit bot <commit-bot@chromium.org> Tue Jun 07 19:08:58 2016 -0700
tree cb788140af3bd1c34c0806e0a11f584719ed5650
parent 4dbbd04314cc0606f8d3bafe515c97e52c180f73

A Mozilla developer ran the µmix static analyzer on its codebase and incidentally found some issues regarding null checks in Skia. This fixes the issues that were found.

Downstream bug report: https://bugzilla.mozilla.org/show_bug.cgi?id=1278452

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2046563007

Review-Url: https://codereview.chromium.org/2046563007

src/utils/win/SkDWriteFontFileStream.cpp

diff --git a/src/utils/win/SkDWriteFontFileStream.cpp b/src/utils/win/SkDWriteFontFileStream.cpp
index 460f90e..6c73441 100644
--- a/src/utils/win/SkDWriteFontFileStream.cpp
+++ b/src/utils/win/SkDWriteFontFileStream.cpp
@@ -139,7 +139,7 @@
                                               SkDWriteFontFileStreamWrapper** streamFontFileStream)
 {
     *streamFontFileStream = new SkDWriteFontFileStreamWrapper(stream);
-    if (nullptr == streamFontFileStream) {
+    if (nullptr == *streamFontFileStream) {
         return E_OUTOFMEMORY;
     }
     return S_OK;