diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2017-12-18 09:54:02 +0000 |
|---|---|---|
| committer | Caolán McNamara <caolanm@redhat.com> | 2017-12-18 22:14:17 +0100 |
| commit | a7c772566bb8ac6a088b808cf0ef550e7b06560b (patch) | |
| tree | 56dd899b4e2081d74ea590e95e70f83f64954c48 /svx/source/gengal | |
| parent | 877ede68f31a25a76c89d47389a190b75df1757d (diff) | |
coverity#1426163 Improper use of negative value
started life as sal_uIntPtr and the various checks are typically
nPos < thing.size() just checking one bound, so sal_uInt32 looks safer
after all
Change-Id: Ibac839fc45b10d96ce4d4c8002fd26b20f70ae1c
Reviewed-on: https://gerrit.libreoffice.org/46697
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
Diffstat (limited to 'svx/source/gengal')
| -rw-r--r-- | svx/source/gengal/gengal.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/svx/source/gengal/gengal.cxx b/svx/source/gengal/gengal.cxx index 430899ef9643..fc8568b9d3cd 100644 --- a/svx/source/gengal/gengal.cxx +++ b/svx/source/gengal/gengal.cxx @@ -122,7 +122,7 @@ static void createTheme( const OUString& aThemeName, const OUString& aGalleryURL fprintf( stderr, "Failed to import '%s'\n", OUStringToOString( aIter->GetMainURL(INetURLObject::DecodeMechanism::NONE), RTL_TEXTENCODING_UTF8 ).getStr() ); else - fprintf( stderr, "Imported file '%s' (%" SAL_PRIdINT32 "u)\n", + fprintf( stderr, "Imported file '%s' (%u)\n", OUStringToOString( aIter->GetMainURL(INetURLObject::DecodeMechanism::NONE), RTL_TEXTENCODING_UTF8 ).getStr(), pGalTheme->GetObjectCount() ); } |