diff options
author | Caolán McNamara <caolanm@redhat.com> | 2018-02-24 20:43:12 +0000 |
---|---|---|
committer | Caolán McNamara <caolanm@redhat.com> | 2018-02-25 01:00:13 +0100 |
commit | 05bd9ed45d7a985ba7a53d0c122d6dbcd6cb2652 (patch) | |
tree | 1e65368cb3e0f7c922a77860924273c1e9cc3f07 /hwpfilter | |
parent | 86a5c40b4ab4cdf20733dbbb1698472c9a0eeded (diff) |
forcepoint #8 ensure ColumnDef lifetime
Change-Id: Idb0c7b1530dc57f4d7c14751f1b76caecc3b03a6
Reviewed-on: https://gerrit.libreoffice.org/50288
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
Diffstat (limited to 'hwpfilter')
-rw-r--r-- | hwpfilter/source/hinfo.cxx | 11 | ||||
-rw-r--r-- | hwpfilter/source/hinfo.h | 2 | ||||
-rw-r--r-- | hwpfilter/source/hpara.cxx | 4 | ||||
-rw-r--r-- | hwpfilter/source/hwpfile.cxx | 6 | ||||
-rw-r--r-- | hwpfilter/source/hwpfile.h | 5 |
5 files changed, 14 insertions, 14 deletions
diff --git a/hwpfilter/source/hinfo.cxx b/hwpfilter/source/hinfo.cxx index 9cf5907e9950..d477d845c3b1 100644 --- a/hwpfilter/source/hinfo.cxx +++ b/hwpfilter/source/hinfo.cxx @@ -201,6 +201,7 @@ ParaShape::ParaShape() , pspacing_next(0) , condense(0) , arrange_type(0) + , xColdef(new ColumnDef) , shade(0) , outline(0) , outline_continue(0) @@ -240,17 +241,17 @@ void ParaShape::Read(HWPFile & hwpf) return; tab.position = tmp16; } - hwpf.Read1b(coldef.ncols); - hwpf.Read1b(coldef.separator); + hwpf.Read1b(xColdef->ncols); + hwpf.Read1b(xColdef->separator); if (!hwpf.Read2b(tmp16)) return; - coldef.spacing = tmp16; + xColdef->spacing = tmp16; if (!hwpf.Read2b(tmp16)) return; - coldef.columnlen = tmp16; + xColdef->columnlen = tmp16; if (!hwpf.Read2b(tmp16)) return; - coldef.columnlen0 = tmp16; + xColdef->columnlen0 = tmp16; hwpf.Read1b(shade); hwpf.Read1b(outline); hwpf.Read1b(outline_continue); diff --git a/hwpfilter/source/hinfo.h b/hwpfilter/source/hinfo.h index 049082e39e28..2cdaba65b2da 100644 --- a/hwpfilter/source/hinfo.h +++ b/hwpfilter/source/hinfo.h @@ -292,7 +292,7 @@ struct ParaShape unsigned char condense; unsigned char arrange_type; TabSet tabs[MAXTABS]; - ColumnDef coldef; + std::shared_ptr<ColumnDef> xColdef; unsigned char shade; unsigned char outline; unsigned char outline_continue; diff --git a/hwpfilter/source/hpara.cxx b/hwpfilter/source/hpara.cxx index ebade2180463..08486c827d08 100644 --- a/hwpfilter/source/hpara.cxx +++ b/hwpfilter/source/hpara.cxx @@ -124,8 +124,8 @@ bool HWPPara::Read(HWPFile & hwpf, unsigned char flag) } if (nch && !reuse_shape){ - if( pshape->coldef.ncols > 1 ) { - hwpf.SetColumnDef(&(pshape->coldef)); + if( pshape->xColdef->ncols > 1 ) { + hwpf.SetColumnDef(pshape->xColdef); } } diff --git a/hwpfilter/source/hwpfile.cxx b/hwpfilter/source/hwpfile.cxx index f7b78fdf704c..552233dd853d 100644 --- a/hwpfilter/source/hwpfile.cxx +++ b/hwpfilter/source/hwpfile.cxx @@ -412,7 +412,7 @@ void HWPFile::TagsRead() ColumnDef *HWPFile::GetColumnDef(int num) { if (static_cast<size_t>(num) < columnlist.size()) - return columnlist[num]->coldef; + return columnlist[num]->xColdef.get(); else return nullptr; } @@ -569,12 +569,12 @@ void HWPFile::AddColumnInfo() setMaxSettedPage(); } -void HWPFile::SetColumnDef(ColumnDef *coldef) +void HWPFile::SetColumnDef(const std::shared_ptr<ColumnDef>& rColdef) { ColumnInfo *cinfo = columnlist.back().get(); if( cinfo->bIsSet ) return; - cinfo->coldef = coldef; + cinfo->xColdef = rColdef; cinfo->bIsSet = true; } diff --git a/hwpfilter/source/hwpfile.h b/hwpfilter/source/hwpfile.h index f39681ffad84..442b442e0514 100644 --- a/hwpfilter/source/hwpfile.h +++ b/hwpfilter/source/hwpfile.h @@ -72,11 +72,10 @@ class HStream; struct ColumnInfo{ int start_page; bool bIsSet; - ColumnDef *coldef; + std::shared_ptr<ColumnDef> xColdef; explicit ColumnInfo(int num){ start_page = num; bIsSet = false; - coldef = nullptr; } }; @@ -211,7 +210,7 @@ class DLLEXPORT HWPFile void AddBox(FBox *); void AddPage(){ m_nCurrentPage++;} void AddColumnInfo(); - void SetColumnDef(ColumnDef *coldef); + void SetColumnDef(std::shared_ptr<ColumnDef> const &); void AddParaShape(std::shared_ptr<ParaShape> const &); void AddCharShape(std::shared_ptr<CharShape> const &); void AddFBoxStyle(FBoxStyle *); |