Avoid undefined signed integer overflow

Change-Id: Idbb8109c36dfe1c8ed4acb8dff1a1538e386abd8

Avoid undefined signed integer overflow
Change-Id: Idbb8109c36dfe1c8ed4acb8dff1a1538e386abd8
1d203f2d · Stephan Bergmann · 7bc00821 · 1d203f2d
Kaydet (Commit) 1d203f2d authored May 28, 2014 tarafından Stephan Bergmann
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 7 deletions

stgdir.cxx sot/source/sdstor/stgdir.cxx +2 -7

No files found.
--- a/sot/source/sdstor/stgdir.cxx
+++ b/sot/source/sdstor/stgdir.cxx
@@ -967,13 +967,8 @@ bool StgDirStrm::Store()

 void* StgDirStrm::GetEntry( sal_Int32 n, bool bDirty )
 {
-    if( n < 0 )
-        return NULL;
-
-    n *= STGENTRY_SIZE;
-    if( n >= nSize )
-        return NULL;
-    return GetPtr( n, true, bDirty );
+    return n < 0 || n >= nSize / STGENTRY_SIZE
+        ? NULL : GetPtr( n * STGENTRY_SIZE, true, bDirty );
 }

 // Find a dir entry.