diff options
author | Caolán McNamara <caolanm@redhat.com> | 2021-09-19 12:46:45 +0100 |
---|---|---|
committer | Caolán McNamara <caolanm@redhat.com> | 2021-09-19 15:07:26 +0200 |
commit | ecab8f30de7d086f5b0f736e4aa77ba038b905b7 (patch) | |
tree | 3e5e27a373cddfde4fdd36e42089db330ce5ad4c | |
parent | 6e0f3a6803c6ab8fda3578e354e1723951388a04 (diff) |
cid#1492159 Uninitialized scalar field
since...
commit c2e6a068c112f4c5866b6371d02362bd71f903a3
Date: Wed Sep 8 12:07:48 2021 +1000
vcl: create FontMetric from PhysicalFontFace
where previously we had...
FontMetric aFontMetric;
...
aFontMetric.SetFamilyName( rData.GetFamilyName() );
...
return aFontMetric;
so the FontMetric default ctor was called which initialized all
FontMetric's members, but afterwards with...
return FontMetric(rData);
the new FontMetric ctor taking a const PhysicalFontFace& arg left all
the non-inherited members of FontMetric uninitialized
Change-Id: I91ee08ed1d63d15eccec44c87841884b2992eacb
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/122317
Tested-by: Caolán McNamara <caolanm@redhat.com>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
-rw-r--r-- | vcl/source/font/fontmetric.cxx | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/vcl/source/font/fontmetric.cxx b/vcl/source/font/fontmetric.cxx index 03996cd7a4d8..93984357d4d0 100644 --- a/vcl/source/font/fontmetric.cxx +++ b/vcl/source/font/fontmetric.cxx @@ -52,6 +52,7 @@ FontMetric::FontMetric() FontMetric::FontMetric( const FontMetric& rFontMetric ) = default; FontMetric::FontMetric(PhysicalFontFace const& rFace) + : FontMetric() { SetFamilyName(rFace.GetFamilyName()); SetStyleName(rFace.GetStyleName()); |