Kaydet (Commit) e40a7f2e authored tarafından Caolán McNamara's avatar Caolán McNamara

coverity#735808 Dereference after null check

Change-Id: Idd8ff57f5c970d15e199831bb1b1251ced32eeec
üst b854cb78
...@@ -273,7 +273,7 @@ sal_Bool ScTabPageSortFields::FillItemSet( SfxItemSet& rArgSet ) ...@@ -273,7 +273,7 @@ sal_Bool ScTabPageSortFields::FillItemSet( SfxItemSet& rArgSet )
// If the "OK" was selected on the Options page while the sort // If the "OK" was selected on the Options page while the sort
// direction was changed, then the first field (i.e. nFieldArr[0]) // direction was changed, then the first field (i.e. nFieldArr[0])
// of the respective direction is chosen as the sorting criterion: // of the respective direction is chosen as the sorting criterion:
if ( bSortByRows != pDlg->GetByRows() ) if ( pDlg && bSortByRows != pDlg->GetByRows() )
{ {
for ( sal_uInt16 i=0; i<nSortKeyCount; i++ ) for ( sal_uInt16 i=0; i<nSortKeyCount; i++ )
aNewSortData.maKeyState[i].nField = ( bSortByRows ? aNewSortData.maKeyState[i].nField = ( bSortByRows ?
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment