pcbnew: Check before reading/writing cache
The open/create calls can fail so we need to account for this before
using the result that assumes success.
(cherry picked from commit 88729de685
)
This commit is contained in:
parent
fcb6f814de
commit
52da7f931b
|
@ -343,12 +343,15 @@ void FOOTPRINT_LIST_IMPL::WriteCacheToFile( wxTextFile* aCacheFile )
|
|||
{
|
||||
if( aCacheFile->Exists() )
|
||||
{
|
||||
aCacheFile->Open();
|
||||
if( !aCacheFile->Open() )
|
||||
return;
|
||||
|
||||
aCacheFile->Clear();
|
||||
}
|
||||
else
|
||||
{
|
||||
aCacheFile->Create();
|
||||
if( !aCacheFile->Create() )
|
||||
return;
|
||||
}
|
||||
|
||||
aCacheFile->AddLine( wxString::Format( "%lld", m_list_timestamp ) );
|
||||
|
@ -376,10 +379,8 @@ void FOOTPRINT_LIST_IMPL::ReadCacheFromFile( wxTextFile* aCacheFile )
|
|||
|
||||
try
|
||||
{
|
||||
if( aCacheFile->Exists() )
|
||||
if( aCacheFile->Exists() && aCacheFile->Open() )
|
||||
{
|
||||
aCacheFile->Open();
|
||||
|
||||
aCacheFile->GetFirstLine().ToLongLong( &m_list_timestamp );
|
||||
|
||||
while( aCacheFile->GetCurrentLine() + 6 < aCacheFile->GetLineCount() )
|
||||
|
|
Loading…
Reference in New Issue