diff options
| author |  djsollen <djsollen@google.com> | 2014-08-28 12:34:41 -0700 |
|---|---|---|
| committer |  Commit bot <commit-bot@chromium.org> | 2014-08-28 12:34:41 -0700 |
| commit | 7ba2828aa6687f9d12f4fa0b0a2ef9a9dc2efa68 (patch) | |
| tree | ff2a86a7df54af1737139142171192b13ec3d6c6 | |
| parent | d21a68632473c014e9d894513d27451173eb6993 (diff) | |
Fix memory leak in SkPDFType1Font
BUG=skia:2880
R=bungeman@google.com
Author: djsollen@google.com
Review URL: https://codereview.chromium.org/514313003
| -rw-r--r-- | src/pdf/SkPDFFont.cpp | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/pdf/SkPDFFont.cpp b/src/pdf/SkPDFFont.cpp index 652b56d706..9e13053c60 100644 --- a/src/pdf/SkPDFFont.cpp +++ b/src/pdf/SkPDFFont.cpp @@ -1312,13 +1312,13 @@ bool SkPDFType1Font::addFontDescriptor(int16_t defaultWidth) { size_t data SK_INIT_TO_AVOID_WARNING; size_t trailer SK_INIT_TO_AVOID_WARNING; SkAutoTUnref<SkStream> rawFontData(typeface()->openStream(&ttcIndex)); - SkData* fontData = handle_type1_stream(rawFontData.get(), &header, &data, - &trailer); - if (fontData == NULL) { + SkAutoTUnref<SkData> fontData(handle_type1_stream(rawFontData.get(), &header, + &data, &trailer)); + if (fontData.get() == NULL) { return false; } if (canEmbed()) { - SkAutoTUnref<SkPDFStream> fontStream(new SkPDFStream(fontData)); + SkAutoTUnref<SkPDFStream> fontStream(new SkPDFStream(fontData.get())); addResource(fontStream.get()); fontStream->insertInt("Length1", header); fontStream->insertInt("Length2", data); |