From 3640475b0a8ea4a2eca060ed0bcf1f8f8da22375 Mon Sep 17 00:00:00 2001 From: Wolfgang Bauer Date: Fri, 4 May 2018 16:22:57 +0200 Subject: [PATCH] Revert "Support NTFS hidden files" This reverts commit d7cce9937d5e9af2753fadb82d11f308b58bb8fa. The new support for NTFS hidden files causes the mountpoint itself to be hidden as well, apparently the root directory of an NTFS partition always has the "hidden" attribute set. https://bugs.kde.org/show_bug.cgi?id=392913 http://bugzilla.opensuse.org/show_bug.cgi?id=1091780 --- src/ioslaves/file/file_unix.cpp | 54 ----------------------------------------- 1 file changed, 54 deletions(-) Index: kio-5.48.0git.20180707T215251~1fa87fdd/src/ioslaves/file/file_unix.cpp =================================================================== --- kio-5.48.0git.20180707T215251~1fa87fdd.orig/src/ioslaves/file/file_unix.cpp +++ kio-5.48.0git.20180707T215251~1fa87fdd/src/ioslaves/file/file_unix.cpp @@ -38,9 +38,6 @@ #include #include -#if HAVE_SYS_XATTR_H -#include -#endif #include #include @@ -416,52 +413,6 @@ static bool isLocalFileSameHost(const QU return (QString::compare(url.host(), QLatin1String(hostname), Qt::CaseInsensitive) == 0); } -#if HAVE_SYS_XATTR_H -static bool isNtfsHidden(const QString &filename) -{ - constexpr auto attrName = "system.ntfs_attrib_be"; - const auto filenameEncoded = QFile::encodeName(filename); -#ifdef Q_OS_MACOS - auto length = getxattr(filenameEncoded.data(), attrName, nullptr, 0, 0, XATTR_NOFOLLOW); -#else - auto length = getxattr(filenameEncoded.data(), attrName, nullptr, 0); -#endif - if (length <= 0) { - return false; - } - constexpr size_t xattr_size = 1024; - char strAttr[xattr_size]; -#ifdef Q_OS_MACOS - length = getxattr(filenameEncoded.data(), attrName, strAttr, xattr_size, 0, XATTR_NOFOLLOW); -#else - length = getxattr(filenameEncoded.data(), attrName, strAttr, xattr_size); -#endif - if (length <= 0) { - return false; - } - - // Decode result to hex string - static constexpr auto digits = "0123456789abcdef"; - QVarLengthArray hexAttr(static_cast(length) * 2 + 4); - char *c = strAttr; - char *e = hexAttr.data(); - *e++ ='0'; - *e++ = 'x'; - for (auto n = 0; n < length; n++, c++) { - *e++ = digits[(static_cast(*c) >> 4)]; - *e++ = digits[(static_cast(*c) & 0x0F)]; - } - *e = '\0'; - - // Decode hex string to int - auto intAttr = static_cast(strtol(hexAttr.data(), nullptr, 16)); - - constexpr auto FILE_ATTRIBUTE_HIDDEN = 0x2u; - return static_cast(intAttr & FILE_ATTRIBUTE_HIDDEN); -} -#endif - - void FileProtocol::listDir(const QUrl &url) { if (!isLocalFileSameHost(url)) { @@ -558,11 +509,6 @@ void FileProtocol::listDir(const QUrl &u } else { if (createUDSEntry(filename, QByteArray(ep->d_name), entry, details)) { -#if HAVE_SYS_XATTR_H - if (isNtfsHidden(filename)) { - entry.fastInsert(KIO::UDSEntry::UDS_HIDDEN, 1); - } -#endif listEntry(entry); } }