From kde-commits  Sun Jan 31 23:36:03 2010
From: Dawit Alemayehu <adawit () kde ! org>
Date: Sun, 31 Jan 2010 23:36:03 +0000
To: kde-commits
Subject: branches/KDE/4.4/kdelibs/kio/kio
Message-Id: <1264980963.438206.23641.nullmailer () svn ! kde ! org>
X-MARC-Message: https://marc.info/?l=kde-commits&m=126498097311942

SVN commit 1083287 by adawit:

Factor out the check for content allowed into its own function and restored the check for the 'data' protocol

 M  +11 -3     accessmanager.cpp  


--- branches/KDE/4.4/kdelibs/kio/kio/accessmanager.cpp #1083286:1083287
@@ -45,6 +45,7 @@
 public:
     AccessManagerPrivate():externalContentAllowed(true) {}
     KIO::MetaData metaDataForRequest(QNetworkRequest request);
+    bool isRequestAllowed(const QUrl& url) const;
 
     bool externalContentAllowed;
     KIO::MetaData requestMetaData;
@@ -122,8 +123,8 @@
 {
     KIO::SimpleJob *kioJob = 0;
 
-    if ( !d->externalContentAllowed && req.url().scheme() != "file" && !req.url().scheme().isEmpty() ) {
-        kDebug( 7044 ) << "Blocked: " << req.url().scheme() <<  req.url();
+    if ( !d->isRequestAllowed(req.url()) ) {
+        kDebug( 7044 ) << "Blocked: " << req.url();
         /* if kioJob equals zero, the AccessManagerReply will block the request */
         return new KDEPrivate::AccessManagerReply(op, req, kioJob, this);
     }
@@ -173,7 +174,6 @@
     return reply;
 }
 
-
 KIO::MetaData AccessManager::AccessManagerPrivate::metaDataForRequest(QNetworkRequest request)
 {
     KIO::MetaData metaData;
@@ -235,7 +235,15 @@
     return metaData;
 }
 
+bool AccessManager::AccessManagerPrivate::isRequestAllowed(const QUrl& url) const
+{
+    const QString scheme (url.scheme());
 
+    return (externalContentAllowed && !scheme.isEmpty() &&
+            scheme != QLatin1String("file")  && scheme != QLatin1String("data"));
+}
+
+
 using namespace KIO::Integration;
 
 CookieJar::CookieJar(QObject* parent)