diff options
author | 2016-02-16 06:59:18 -0800 | |
---|---|---|
committer | 2016-02-16 06:59:18 -0800 | |
commit | 9dec5d2acdd12202a1f0ed571cb5ecee574a2550 (patch) | |
tree | 5fb47683c5b261c85f88a54f4f29732059e832c1 /src | |
parent | ddd22d86455d95941ea21253b210819c1d4a3863 (diff) |
Make SkRWBuffer destruct safely
Check for NULL before calling unref.
Add a test.
BUG=483369
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1691383002
Review URL: https://codereview.chromium.org/1691383002
Diffstat (limited to 'src')
-rw-r--r-- | src/core/SkRWBuffer.cpp | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/core/SkRWBuffer.cpp b/src/core/SkRWBuffer.cpp index 0784378d99..c0a93bdf53 100644 --- a/src/core/SkRWBuffer.cpp +++ b/src/core/SkRWBuffer.cpp @@ -168,7 +168,9 @@ SkRWBuffer::SkRWBuffer(size_t initialCapacity) : fHead(nullptr), fTail(nullptr), SkRWBuffer::~SkRWBuffer() { this->validate(); - fHead->unref(); + if (fHead) { + fHead->unref(); + } } void SkRWBuffer::append(const void* src, size_t length) { |