diff options
| author | Miklos Vajna <vmiklos@collabora.co.uk> | 2014-06-02 20:14:16 +0200 |
|---|---|---|
| committer | Miklos Vajna <vmiklos@collabora.co.uk> | 2014-06-02 20:29:00 +0200 |
| commit | 6092e2eba3f74c9632f7862b2368b0fcf7732f85 (patch) | |
| tree | 82d1cce1fcdb74f83cdc408e668b5ab5e5305a52 | |
| parent | c7c14abfd870d3849e74d379057503f16728a392 (diff) | |
fdo#79384 RTF import: allow { without } in style names
This is not valid, but LO 3.4 supported it, so add a workaround...
Change-Id: Ib8eefdf41fe9801db6f7e5873eaa1436518912fb
| -rw-r--r-- | sw/qa/core/data/rtf/pass/fdo79384.rtf | 5 | ||||
| -rw-r--r-- | writerfilter/source/rtftok/rtfdocumentimpl.cxx | 7 |
2 files changed, 11 insertions, 1 deletions
diff --git a/sw/qa/core/data/rtf/pass/fdo79384.rtf b/sw/qa/core/data/rtf/pass/fdo79384.rtf new file mode 100644 index 000000000000..84875a96e044 --- /dev/null +++ b/sw/qa/core/data/rtf/pass/fdo79384.rtf @@ -0,0 +1,5 @@ +{\rtf1
+{\stylesheet
+{\*\cs35\snext35\hich\af5\dbch\af5\loch\f5 Mp{u y{p;}
+}
+\par }
diff --git a/writerfilter/source/rtftok/rtfdocumentimpl.cxx b/writerfilter/source/rtftok/rtfdocumentimpl.cxx index cc99fbc8f05e..079f43ee5b1c 100644 --- a/writerfilter/source/rtftok/rtfdocumentimpl.cxx +++ b/writerfilter/source/rtftok/rtfdocumentimpl.cxx @@ -956,7 +956,12 @@ int RTFDocumentImpl::resolveChars(char ch) bool bUnicodeChecked = false; bool bSkipped = false; - while (!Strm().IsEof() && (m_aStates.top().nInternalState == INTERNAL_HEX || (ch != '{' && ch != '}' && ch != '\\'))) + + // Workaround for buggy input: if we're inside a style entry, then ignore + // the fact that '{' without a matching '}' is invalid. + bool bStyleEntry = m_aStates.top().nDestinationState == DESTINATION_STYLEENTRY; + + while (!Strm().IsEof() && (m_aStates.top().nInternalState == INTERNAL_HEX || ((ch != '{' || bStyleEntry) && ch != '}' && ch != '\\'))) { if (m_aStates.top().nInternalState == INTERNAL_HEX || (ch != 0x0d && ch != 0x0a)) { |