Nikolay Sivov : dwrite: Check GetGdiCompatibleMetrics() return value ( Coverity).
Alexandre Julliard
julliard at wine.codeweavers.com
Mon Jul 13 09:30:03 CDT 2015
Module: wine
Branch: master
Commit: cf5d9848d800706cd2ba8dc5544f0414f0ac8f37
URL: http://source.winehq.org/git/wine.git/?a=commit;h=cf5d9848d800706cd2ba8dc5544f0414f0ac8f37
Author: Nikolay Sivov <nsivov at codeweavers.com>
Date: Sun Jul 12 23:35:10 2015 +0300
dwrite: Check GetGdiCompatibleMetrics() return value (Coverity).
---
dlls/dwrite/analyzer.c | 7 ++++++-
dlls/dwrite/layout.c | 5 +++--
2 files changed, 9 insertions(+), 3 deletions(-)
diff --git a/dlls/dwrite/analyzer.c b/dlls/dwrite/analyzer.c
index 8c2a801..3417109 100644
--- a/dlls/dwrite/analyzer.c
+++ b/dlls/dwrite/analyzer.c
@@ -1090,7 +1090,12 @@ static HRESULT WINAPI dwritetextanalyzer_GetGdiCompatibleGlyphPlacements(IDWrite
return hr;
}
- IDWriteFontFace_GetGdiCompatibleMetrics(fontface, emSize, pixels_per_dip, transform, &metrics);
+ hr = IDWriteFontFace_GetGdiCompatibleMetrics(fontface, emSize, pixels_per_dip, transform, &metrics);
+ if (FAILED(hr)) {
+ IDWriteFontFace1_Release(fontface1);
+ WARN("failed to get compat metrics, 0x%08x\n", hr);
+ return hr;
+ }
for (i = 0; i < glyph_count; i++) {
INT32 a;
diff --git a/dlls/dwrite/layout.c b/dlls/dwrite/layout.c
index 8b4c2cd..70e2095 100644
--- a/dlls/dwrite/layout.c
+++ b/dlls/dwrite/layout.c
@@ -828,12 +828,13 @@ static HRESULT layout_compute_runs(struct dwrite_textlayout *layout)
/* baseline derived from font metrics */
if (layout->gdicompatible) {
- /* FIXME: check return value when it's actually implemented */
- IDWriteFontFace_GetGdiCompatibleMetrics(run->run.fontFace,
+ hr = IDWriteFontFace_GetGdiCompatibleMetrics(run->run.fontFace,
run->run.fontEmSize,
layout->pixels_per_dip,
&layout->transform,
&fontmetrics);
+ if (FAILED(hr))
+ WARN("failed to get compat metrics, 0x%08x\n", hr);
}
else
IDWriteFontFace_GetMetrics(run->run.fontFace, &fontmetrics);
More information about the wine-cvs
mailing list