diff options
author | Stephan Bergmann <sbergman@redhat.com> | 2021-07-23 12:20:05 +0200 |
---|---|---|
committer | Stephan Bergmann <sbergman@redhat.com> | 2021-07-23 14:03:11 +0200 |
commit | daffec714ef28f52ae2d70e5553500046b8b26c2 (patch) | |
tree | 13227644832196633dcb5f9538c9fd7a5f78fa98 /ure | |
parent | 252e403213bf2a1c1b1f7b07f1dd647b450cb312 (diff) |
Avoid strtol on non-NUL-terminated input
...as could be observed with ASan and ASAN_OPTIONS=strict_string_checks=1 during
e.g. UITest_conditional_format.
strtol supports leading spaces, sign, and "0x", but none of that appears to be
relevant here, so readInt32 doesn't support that.
Hdf::m_pItData was redundant and always pointed at m_aItData.data(), and has
been elided to make it more obvious that the places that used it, which now also
need to use m_aItData.size(), actually use m_aItData.
Change-Id: I3d4f625cd5836438271d7c1a8d2ae06f0a45a37c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119403
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Diffstat (limited to 'ure')
0 files changed, 0 insertions, 0 deletions