https://bugs.kde.org/show_bug.cgi?id=485507

--- Comment #6 from Marcel Hasler <mahas...@gmail.com> ---
Looking at the code some more, it seems that many functions explicitly check
for both cases, e.g. in StorageAccess::filePath():

    const QString path = clearTextPath();
    if (path.isEmpty() || path == "/") {
        QString();
    }

Other functions, such as StorageAccess::setup() (which I think might be the
culprit here), don't:

    if (m_device->isEncryptedContainer() && clearTextPath().isEmpty()) {
        return requestPassphrase();
    } else {
        return mount();
    }

Perhaps it would make sense to refactor this check into an own function (e.g.
StorageAccess::isUnlocked()).

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to