To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=106430 Issue #|106430 Summary|binfilter: sch_docshell.cxx: 2 * possible null pointer | dereference Component|framework Version|DEV300m63 Platform|All URL| OS/Version|Linux Status|UNCONFIRMED Status whiteboard| Keywords| Resolution| Issue type|DEFECT Priority|P3 Subcomponent|code Assigned to|mba Reported by|dcb314
------- Additional comments from dcb...@openoffice.org Thu Oct 29 11:34:42 +0000 2009 ------- For source code file binfilter/bf_sch/source/ui/docshell/sch_docshell.cxx, around line 641 is source code long nFileFormat = pStor->GetVersion(); but the very next use of pStor, a little later, is if( pStor ) So the first line assumes that pStor is legal, when the second line indicates that it might not be. Suggest code rework. Maybe a test of pStor before first use would be enough. The same thing happens around line 787 of the same file, function SchChartDocShell::SaveAs, same local variable name pStor. --------------------------------------------------------------------- Please do not reply to this automatically generated notification from Issue Tracker. Please log onto the website and enter your comments. http://qa.openoffice.org/issue_handling/project_issues.html#notification --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@framework.openoffice.org For additional commands, e-mail: issues-h...@framework.openoffice.org --------------------------------------------------------------------- To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org For additional commands, e-mail: allbugs-h...@openoffice.org