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

Andrius Štikonas <andr...@stikonas.eu> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ch.ehrlic...@gmx.de

--- Comment #2 from Andrius Štikonas <andr...@stikonas.eu> ---
Hmm, there doesn't seem to be canonical way to decode blkid strings in Qt.

Qt has a internal function to do that
https://github.com/qt/qtbase/blob/9a320b037ce7b63c5f9de6fbb4e391612e857a95/src/corelib/io/qstorageinfo_unix.cpp#L456
which we could presumably "steal" it.

But tt seems that it incorrectly decodes backslashes...

Christian, would you happen to remember why backslashes are ignored in
qstorageinfo_unix.cpp::decodeFsEncString
(// but not the backslash to prevent collisions)

Because of that conditional, backslash is not decoded and it does seem correct
to me. For example, I created a label called
rootfs test\test
blkid then encodes it to rootfs\x20test\x5ctest
which your function decoded to rootfs test\x5ctest

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

Reply via email to