diff options
author | Caolán McNamara <caolanm@redhat.com> | 2022-08-09 11:38:47 +0100 |
---|---|---|
committer | Xisco Fauli <xiscofauli@libreoffice.org> | 2022-08-10 09:11:07 +0200 |
commit | b30545c75675b2d414f9aae175265acc9cdcf6cd (patch) | |
tree | 9d16021d0c39b658313dca185b3081c57640b3e1 | |
parent | 97f687074db5d4972af9b163047aa233f7617a42 (diff) |
crashreporting: frequent crash seen in PackedFile::flush
at:
if (osl::File::E_None == osl::FileBase::createTempFile(nullptr, &aHandle, &aTempURL))
{
....
if (osl_File_E_None == osl_writeFile(aHandle, ...
createTempFile is returning osl::File::E_None but presumably we are
missing setting a possible error state here
https: //crashreport.libreoffice.org/stats/crash_details/ea4b4050-dd0f-42b1-b30f-b67612806371
Change-Id: Ib2d0c3c91e40fe985571e79822b91f2faf401471
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138022
Tested-by: Jenkins
Reviewed-by: Xisco Fauli <xiscofauli@libreoffice.org>
-rw-r--r-- | sal/osl/w32/tempfile.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sal/osl/w32/tempfile.cxx b/sal/osl/w32/tempfile.cxx index a4c272e57768..5d5d758ce244 100644 --- a/sal/osl/w32/tempfile.cxx +++ b/sal/osl/w32/tempfile.cxx @@ -192,7 +192,7 @@ oslFileError SAL_CALL osl_createTempFile( if (tmp_name) { - osl_createTempFile_impl_( + osl_error = osl_createTempFile_impl_( base_directory, tmp_name, b_delete_on_close, |