ofz#53016 Undefined-shift

before commit 4b95451f859bac8e05956ce12df17f1ee410032d
Date:   Wed Sep 21 20:54:00 2022 +0200

    split utl::TempFile into fast and named variants

    which makes it easier to know what each variant requires
    to stay on it's happy path

we closed the writable stream and got another readonly stream, after
that commit we have a read-write stream. With the "true" arg to
Storage it will Commit back to the underlying storage its contents
in its dtor. With a ReadOnly stream this failed and nothing happened,
with a read-write stream there is a write which didn't happen with a
read-only one.

Lets just use false here, we're not interested in keeping the contents.

"Direct" in this context is presumably analogous to the Compound
File Format "Direct mode":

"The mode in which files are accessed, either transacted or direct,
determines when changes are committed.

    Transacted mode uses a two-phase commit operation to make changes to
objects in a compound file, thereby keeping both the old and the new
copies of the document available until the user chooses to either save
or undo the changes.

    Direct mode incorporates changes to the document as they are made,
without the ability to later undo them.

For more information about access modes, see the OLE Programmer's Reference."

Change-Id: I1f710de227807073a6aff11a86d7cfaf6d380509
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/142283
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>

1 files changed, 1 insertions, 1 deletions

diff --git a/filter/source/msfilter/svdfppt.cxx b/filter/source/msfilter/svdfppt.cxx
index 3020083756a0..cb6381fb6fd3 100644
--- a/filter/source/msfilter/svdfppt.cxx
+++ b/filter/source/msfilter/svdfppt.cxx
@@ -1855,7 +1855,7 @@ rtl::Reference<SdrObject> SdrPowerPointImport::ImportOLE( sal_uInt32 nOLEId,
             if ( bSuccess )
             {
                 pDest->Seek(0);
-                Storage* pObjStor = new Storage(*pDest, true);
+                Storage* pObjStor = new Storage(*pDest, false);
                 tools::SvRef<SotStorage> xObjStor( new SotStorage( pObjStor ) );
                 if ( xObjStor.is() && !xObjStor->GetError() )
                 {