diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2019-07-22 12:05:37 +0100 |
|---|---|---|
| committer | Caolán McNamara <caolanm@redhat.com> | 2019-07-23 09:33:28 +0200 |
| commit | e468a063ceafc3941c9570bf67a5dcdaec84254e (patch) | |
| tree | a491149bd23a478342dc6d20b82865a7fb81ae6a /accessibility | |
| parent | 6e207aeb485e2c9580741b0ecf5da73ed79b6d90 (diff) | |
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>
Diffstat (limited to 'accessibility')
| -rw-r--r-- | accessibility/source/extended/AccessibleBrowseBoxTableCell.cxx | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/accessibility/source/extended/AccessibleBrowseBoxTableCell.cxx b/accessibility/source/extended/AccessibleBrowseBoxTableCell.cxx index d622f38f0ab7..193463df6648 100644 --- a/accessibility/source/extended/AccessibleBrowseBoxTableCell.cxx +++ b/accessibility/source/extended/AccessibleBrowseBoxTableCell.cxx @@ -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 ) ); } |