diff --git a/src/modules/partition/core/PartUtils.cpp b/src/modules/partition/core/PartUtils.cpp index 875de3a63..dac76ca17 100644 --- a/src/modules/partition/core/PartUtils.cpp +++ b/src/modules/partition/core/PartUtils.cpp @@ -22,7 +22,11 @@ #include "core/DeviceModel.h" #include "core/KPMHelpers.h" +#include "core/PartitionIterator.h" +#include +#include +#include #include #include @@ -325,4 +329,95 @@ runOsprober( PartitionCoreModule* core ) } +/** + * Does the given @p device contain the root filesystem? This is true if + * the device contains a partition which is currently mounted at / . + */ +static bool +hasRootPartition( Device* device ) +{ + for ( auto it = PartitionIterator::begin( device ); it != PartitionIterator::end( device ); ++it ) + if ( ( *it )->mountPoint() == "/" ) + return true; + return false; } + +static bool +isMounted( Device* device ) +{ + cDebug() << "Checking for mounted partitions in" << device->deviceNode(); + for ( auto it = PartitionIterator::begin( device ); it != PartitionIterator::end( device ); ++it ) + { + cDebug() << " .." << ( *it )->partitionPath() << "mount" << ( *it )->mountPoint(); + if ( ! ( *it )->mountPoint().isEmpty() ) + return true; + } + return false; +} + +static bool +isIso9660( const Device* device ) +{ + QString path = device->deviceNode(); + if ( path.isEmpty() ) + return false; + + QProcess blkid; + blkid.start( "blkid", { path } ); + blkid.waitForFinished(); + QString output = QString::fromLocal8Bit( blkid.readAllStandardOutput() ); + if ( output.contains( "iso9660" ) ) + return true; + + if ( device->partitionTable() && + !device->partitionTable()->children().isEmpty() ) + { + for ( const Partition* partition : device->partitionTable()->children() ) + { + path = partition->partitionPath(); + blkid.start( "blkid", { path } ); + blkid.waitForFinished(); + QString output = QString::fromLocal8Bit( blkid.readAllStandardOutput() ); + if ( output.contains( "iso9660" ) ) + return true; + } + } + return false; +} + + +QList< Device* > getDevices( bool writableOnly ) +{ + using DeviceList = QList< Device* >; + + CoreBackend* backend = CoreBackendManager::self()->backend(); + DeviceList devices = backend->scanDevices( true ); + + cDebug() << "Winnowing" << devices.count() << "devices."; + + // Remove the device which contains / from the list + for ( DeviceList::iterator it = devices.begin(); it != devices.end(); ) + if ( ! ( *it ) || + ( *it )->deviceNode().startsWith( "/dev/zram" ) + ) + { + cDebug() << " .. Winnowing" << ( ( *it ) ? ( *it )->deviceNode() : QString( "" ) ); + it = devices.erase( it ); + + } + else if ( writableOnly && ( + hasRootPartition( *it ) || + isIso9660( *it ) || + isMounted( *it ) ) + ) + { + cDebug() << " .. Winnowing" << ( ( *it ) ? ( *it )->deviceNode() : QString( "" ) ); + it = devices.erase( it ); + } + else + ++it; + + return devices; +} + +} // nmamespace PartUtils diff --git a/src/modules/partition/core/PartUtils.h b/src/modules/partition/core/PartUtils.h index 21d995965..cd289e046 100644 --- a/src/modules/partition/core/PartUtils.h +++ b/src/modules/partition/core/PartUtils.h @@ -23,6 +23,7 @@ #include +class Device; class PartitionCoreModule; class Partition; @@ -62,6 +63,14 @@ bool canBeResized( PartitionCoreModule* core, const QString& partitionPath ); */ OsproberEntryList runOsprober( PartitionCoreModule* core ); +/** + * @brief Gets a list of storage devices. + * @param writableOnly if set to true, only devices which can be overwritten + * safely are returned (e.g. RO-media are ignored, as are mounted partitions). + * @return a list of Devices meeting this criterium. + */ +QList< Device* > getDevices( bool writableOnly = false ); + } #endif // PARTUTILS_H diff --git a/src/modules/partition/core/PartitionCoreModule.cpp b/src/modules/partition/core/PartitionCoreModule.cpp index fde4c847f..752b7753c 100644 --- a/src/modules/partition/core/PartitionCoreModule.cpp +++ b/src/modules/partition/core/PartitionCoreModule.cpp @@ -55,62 +55,6 @@ #include #include -/** - * Does the given @p device contain the root filesystem? This is true if - * the device contains a partition which is currently mounted at / . - */ -static bool -hasRootPartition( Device* device ) -{ - for ( auto it = PartitionIterator::begin( device ); it != PartitionIterator::end( device ); ++it ) - if ( ( *it )->mountPoint() == "/" ) - return true; - return false; -} - -static bool -isMounted( Device* device ) -{ - cDebug() << "Checking for mounted partitions in" << device->deviceNode(); - for ( auto it = PartitionIterator::begin( device ); it != PartitionIterator::end( device ); ++it ) - { - cDebug() << " .." << ( *it )->partitionPath() << "mount" << ( *it )->mountPoint(); - if ( ! ( *it )->mountPoint().isEmpty() ) - return true; - } - return false; -} - -static bool -isIso9660( const Device* device ) -{ - QString path = device->deviceNode(); - if ( path.isEmpty() ) - return false; - - QProcess blkid; - blkid.start( "blkid", { path } ); - blkid.waitForFinished(); - QString output = QString::fromLocal8Bit( blkid.readAllStandardOutput() ); - if ( output.contains( "iso9660" ) ) - return true; - - if ( device->partitionTable() && - !device->partitionTable()->children().isEmpty() ) - { - for ( const Partition* partition : device->partitionTable()->children() ) - { - path = partition->partitionPath(); - blkid.start( "blkid", { path } ); - blkid.waitForFinished(); - QString output = QString::fromLocal8Bit( blkid.readAllStandardOutput() ); - if ( output.contains( "iso9660" ) ) - return true; - } - } - return false; -} - //- DeviceInfo --------------------------------------------- PartitionCoreModule::DeviceInfo::DeviceInfo( Device* _device ) : device( _device ) @@ -171,26 +115,7 @@ PartitionCoreModule::doInit() FileSystemFactory::init(); using DeviceList = QList< Device* >; - - CoreBackend* backend = CoreBackendManager::self()->backend(); - DeviceList devices = backend->scanDevices( true ); - - cDebug() << "Winnowing" << devices.count() << "devices."; - - // Remove the device which contains / from the list - for ( DeviceList::iterator it = devices.begin(); it != devices.end(); ) - if ( ! ( *it ) || - hasRootPartition( *it ) || - isIso9660( *it ) || - isMounted( *it ) || - ( *it )->deviceNode().startsWith( "/dev/zram" ) - ) - { - cDebug() << " .. Winnowing" << ( ( *it ) ? ( *it )->deviceNode() : QString( "" ) ); - it = devices.erase( it ); - } - else - ++it; + DeviceList devices = PartUtils::getDevices( true ); cDebug() << "LIST OF DETECTED DEVICES:"; cDebug() << "node\tcapacity\tname\tprettyName";