elB4RTO/LogDoctor
elB4RTO/LogDoctor
2
2
Fork 0
Code Issues 5 Pull requests Projects 1 Releases 9 Wiki Activity

Added available memory check before parsing

Browse source 
Introduced check on selected logs' size to not exceed available memory
This commit is contained in:
Valentino Orlandi 2023-04-30 20:36:15 +02:00
parent 2039d37c26
commit 8b918c4dc4
Signed by: elB4RTO
GPG key ID: 1719E976DB2D4E71
1 changed files with 22 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

22
logdoctor/modules/craplog/craplog.cpp
View file

@ -4,6 +4,7 @@
#include "utilities/checks.h"
#include "utilities/gzip.h"
#include "utilities/io.h"
#include "utilities/memory.h"
#include "utilities/printables.h"
#include "utilities/strings.h"
#include "utilities/vectors.h"
@ -689,6 +690,7 @@ const bool Craplog::checkStuff()
}
}
size_t logs_size{ 0ul };
for ( const LogFile& file : this->logs_list ) {
if ( ! this->proceed ) { break; }
@ -795,14 +797,34 @@ const bool Craplog::checkStuff()
std::make_tuple( file.path(), file.hash() )
);
this->used_files_hashes.push_back( file.hash() );
logs_size += file.size();
}
// check if there are enough files to use
if ( this->proceed && this->log_files_to_use.size() == 0ul ) {
// no files left, abort
DialogSec::msgNoFileToParse();
this->proceed &= false;
}
// check if the total size of the files do not exceed the available RAM
if ( this->proceed && logs_size >= MemOps::availableMemory() ) {
// no files left, abort
QString msg;
if ( this->dialogs_level >= 1 ) {
msg += QString("\n\n%1: %2").arg(
DialogSec::tr("Available memory"),
PrintSec::printableSize( MemOps::availableMemory() ) );
if ( this->dialogs_level == 2 ) {
msg += QString("\n%1: %2").arg(
DialogSec::tr("Size of the logs"),
PrintSec::printableSize( logs_size ) );
}
}
DialogSec::msgNotEnoughMemory( msg );
this->proceed &= false;
}
return this->proceed;
}