orcmid <orc...@apache.org> has asked orcmid <orc...@apache.org> for review:
Issue 126635: Possible null pointer dereference
https://bz.apache.org/ooo/show_bug.cgi?id=126635

Attachment 85136: patch to main/sw/source/core/doc/poolfmt.cxx
https://bz.apache.org/ooo/attachment.cgi?id=85136&action=edit



--- Comment #4 from orcmid <orc...@apache.org> ---
Created attachment 85136
  --> https://bz.apache.org/ooo/attachment.cgi?id=85136&action=edit
patch to main/sw/source/core/doc/poolfmt.cxx

In the unpatched version, the misplaced check on pColl for NULL caused 
pColl->SetNextTxtFmtColl( ) to never be executed.

In this patch, the final operation is executed when pColl != NULL.

This may cause unexpected behavior if the failure of this operation is
compensated for anywhere else in the code.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@openoffice.apache.org
For additional commands, e-mail: dev-h...@openoffice.apache.org

Reply via email to