cid#1448398 Dereference before null check

Change-Id: I55bddd8f152f34919e9818048aaf2a77a94ccaf0 Reviewed-on: https://gerrit.libreoffice.org/76130 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2019-07-23 09:33:28 +02:00
parent 6e207aeb48
commit e468a063ce
1 changed files with 3 additions and 3 deletions
@@ -119,13 +119,13 @@ namespace accessibility
        SolarMethodGuard aGuard(getMutex());
        ensureIsAlive();

-        if ( !implIsValidIndex( nIndex, implGetText().getLength() ) )
-            throw IndexOutOfBoundsException();
-
        css::awt::Rectangle aRect;

        if ( mpBrowseBox )
        {
+            if ( !implIsValidIndex( nIndex, implGetText().getLength() ) )
+                throw IndexOutOfBoundsException();
+
            aRect = AWTRectangle( mpBrowseBox->GetFieldCharacterBounds( getRowPos(), getColumnPos(), nIndex ) );
        }