summaryrefslogtreecommitdiff
path: root/core
diff options
context:
space:
mode:
authorRyan Harrison <rharrison@chromium.org>2017-09-27 10:58:52 -0400
committerChromium commit bot <commit-bot@chromium.org>2017-09-27 16:12:07 +0000
commitbacf75eeeb5d89fe60b2c77584350ca1b1432bd5 (patch)
treebbf956a86b103f5e470338e585ce8b924b08cd8b /core
parent875e98c581952478f3a3ccef9b2f2e3ed06c5346 (diff)
downloadpdfium-bacf75eeeb5d89fe60b2c77584350ca1b1432bd5.tar.xz
Replace FX_SAFE_STRSIZE with FX_SAFE_SIZE_T
BUG=pdfium:828 Change-Id: If9560338c456b28f968e743a90a3629791d87df0 Reviewed-on: https://pdfium-review.googlesource.com/14832 Commit-Queue: Ryan Harrison <rharrison@chromium.org> Reviewed-by: dsinclair <dsinclair@chromium.org> Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>
Diffstat (limited to 'core')
-rw-r--r--core/fxcrt/bytestring.cpp4
-rw-r--r--core/fxcrt/cfx_binarybuf.cpp2
-rw-r--r--core/fxcrt/fx_safe_types.h1
-rw-r--r--core/fxcrt/widestring.cpp4
-rw-r--r--core/fxcrt/xml/cfx_xmlsyntaxparser.cpp2
5 files changed, 6 insertions, 7 deletions
diff --git a/core/fxcrt/bytestring.cpp b/core/fxcrt/bytestring.cpp
index bd195dda1d..3079eb7b72 100644
--- a/core/fxcrt/bytestring.cpp
+++ b/core/fxcrt/bytestring.cpp
@@ -125,7 +125,7 @@ ByteString::ByteString(const ByteStringView& stringSrc) {
}
ByteString::ByteString(const ByteStringView& str1, const ByteStringView& str2) {
- FX_SAFE_STRSIZE nSafeLen = str1.GetLength();
+ FX_SAFE_SIZE_T nSafeLen = str1.GetLength();
nSafeLen += str2.GetLength();
size_t nNewLen = nSafeLen.ValueOrDie();
@@ -139,7 +139,7 @@ ByteString::ByteString(const ByteStringView& str1, const ByteStringView& str2) {
}
ByteString::ByteString(const std::initializer_list<ByteStringView>& list) {
- FX_SAFE_STRSIZE nSafeLen = 0;
+ FX_SAFE_SIZE_T nSafeLen = 0;
for (const auto& item : list)
nSafeLen += item.GetLength();
diff --git a/core/fxcrt/cfx_binarybuf.cpp b/core/fxcrt/cfx_binarybuf.cpp
index d1bd05385d..3673fa58d4 100644
--- a/core/fxcrt/cfx_binarybuf.cpp
+++ b/core/fxcrt/cfx_binarybuf.cpp
@@ -49,7 +49,7 @@ void CFX_BinaryBuf::EstimateSize(size_t size, size_t step) {
}
void CFX_BinaryBuf::ExpandBuf(size_t add_size) {
- FX_SAFE_STRSIZE new_size = m_DataSize;
+ FX_SAFE_SIZE_T new_size = m_DataSize;
new_size += add_size;
if (m_AllocSize >= new_size.ValueOrDie())
return;
diff --git a/core/fxcrt/fx_safe_types.h b/core/fxcrt/fx_safe_types.h
index aae608096f..51eb89cf76 100644
--- a/core/fxcrt/fx_safe_types.h
+++ b/core/fxcrt/fx_safe_types.h
@@ -14,6 +14,5 @@ typedef pdfium::base::CheckedNumeric<uint32_t> FX_SAFE_UINT32;
typedef pdfium::base::CheckedNumeric<int32_t> FX_SAFE_INT32;
typedef pdfium::base::CheckedNumeric<size_t> FX_SAFE_SIZE_T;
typedef pdfium::base::CheckedNumeric<FX_FILESIZE> FX_SAFE_FILESIZE;
-typedef pdfium::base::CheckedNumeric<size_t> FX_SAFE_STRSIZE;
#endif // CORE_FXCRT_FX_SAFE_TYPES_H_
diff --git a/core/fxcrt/widestring.cpp b/core/fxcrt/widestring.cpp
index a4632ee9db..b9d6dc2080 100644
--- a/core/fxcrt/widestring.cpp
+++ b/core/fxcrt/widestring.cpp
@@ -314,7 +314,7 @@ WideString::WideString(const WideStringView& stringSrc) {
}
WideString::WideString(const WideStringView& str1, const WideStringView& str2) {
- FX_SAFE_STRSIZE nSafeLen = str1.GetLength();
+ FX_SAFE_SIZE_T nSafeLen = str1.GetLength();
nSafeLen += str2.GetLength();
size_t nNewLen = nSafeLen.ValueOrDie();
@@ -328,7 +328,7 @@ WideString::WideString(const WideStringView& str1, const WideStringView& str2) {
}
WideString::WideString(const std::initializer_list<WideStringView>& list) {
- FX_SAFE_STRSIZE nSafeLen = 0;
+ FX_SAFE_SIZE_T nSafeLen = 0;
for (const auto& item : list)
nSafeLen += item.GetLength();
diff --git a/core/fxcrt/xml/cfx_xmlsyntaxparser.cpp b/core/fxcrt/xml/cfx_xmlsyntaxparser.cpp
index 2d1d3d7a08..d72e7afa1d 100644
--- a/core/fxcrt/xml/cfx_xmlsyntaxparser.cpp
+++ b/core/fxcrt/xml/cfx_xmlsyntaxparser.cpp
@@ -106,7 +106,7 @@ CFX_XMLSyntaxParser::CFX_XMLSyntaxParser(
pdfium::base::checked_cast<size_t>(m_pStream->GetLength()));
m_iCurrentPos = m_pStream->GetBOMLength();
- FX_SAFE_STRSIZE alloc_size_safe = m_iXMLPlaneSize;
+ FX_SAFE_SIZE_T alloc_size_safe = m_iXMLPlaneSize;
alloc_size_safe += 1; // For NUL.
if (!alloc_size_safe.IsValid() || alloc_size_safe.ValueOrDie() <= 0) {
m_syntaxParserResult = FX_XmlSyntaxResult::Error;