webengine/osswebengine/WebKit/s60/plugins/PluginHandler.cpp
changeset 5 10e98eab6f85
parent 1 7c90e6132015
child 10 a359256acfc6
--- a/webengine/osswebengine/WebKit/s60/plugins/PluginHandler.cpp	Fri May 08 08:25:06 2009 +0300
+++ b/webengine/osswebengine/WebKit/s60/plugins/PluginHandler.cpp	Fri Jul 03 15:54:40 2009 +0100
@@ -533,43 +533,50 @@
 // -----------------------------------------------------------------------------
 //
 TInt PluginHandler::findPlugin(const TDesC& mimeType) 
-{    
+   {    
     LOAD_PLUGINS
-    
-	HBufC*   newMimeType = NULL;
-	_LIT(KSeparator, "*");
-	TInt  pluginIndex;
+
+    TInt bufSize( KMimeTypeLength > mimeType.Length() ? KMimeTypeLength : mimeType.Length() );
+    HBufC* newMimeType = HBufC::New(bufSize);
+    if( newMimeType == NULL )
+        {
+        return KErrNoMemory;
+        }
+    _LIT(KSeparator, "*");
+    TInt  pluginIndex;
     TInt  mimeIndex;
 
-	if (mimeType.FindF(KMimeTypeAudio) != KErrNotFound){
-		newMimeType = HBufC::NewLC(KMimeTypeLength);
+    if( mimeType.FindF(KMimeTypeAudio) != KErrNotFound )
+        {
         newMimeType->Des().Copy(KMimeTypeAudio);
-		newMimeType->Des().Append(KSeparator);
-	}
-	else if (mimeType.FindF(KMimeTypeVideo) != KErrNotFound){
-		newMimeType = HBufC::NewLC(KMimeTypeLength);
+        newMimeType->Des().Append(KSeparator);
+        }
+    else if ( mimeType.FindF(KMimeTypeVideo) != KErrNotFound )
+        {
         newMimeType->Des().Copy(KMimeTypeVideo);
-		newMimeType->Des().Append(KSeparator);
-	}
-	else{
-		newMimeType = HBufC::NewLC(mimeType.Length());
-        newMimeType = mimeType.AllocL();
-	}
-	CleanupStack::Pop(); //pop newMimeType
-	
-    for (pluginIndex = 0; pluginIndex < m_pluginInfoArray.Count(); pluginIndex++) {
+        newMimeType->Des().Append(KSeparator);
+        }
+    else
+        {
+        newMimeType->Des().Copy(mimeType);
+        }
+
+    for (pluginIndex = 0; pluginIndex < m_pluginInfoArray.Count(); pluginIndex++) 
+        {
         for (mimeIndex = 0;
              mimeIndex < m_pluginInfoArray[pluginIndex]->m_mimeTypes.Count();
-             mimeIndex++) {
-            if (!m_pluginInfoArray[pluginIndex]->m_mimeTypes[mimeIndex]->CompareF(*newMimeType)) {
+             mimeIndex++) 
+            {
+            if (!m_pluginInfoArray[pluginIndex]->m_mimeTypes[mimeIndex]->CompareF(*newMimeType))
+                {
                 delete newMimeType;
-				return m_pluginInfoArray[pluginIndex]->m_handle;
+                return m_pluginInfoArray[pluginIndex]->m_handle;
+                }
             }
         }
+    delete newMimeType;
+    return KErrNotFound;
     }
-	delete newMimeType;
-    return KErrNotFound;
-}
 
 // -----------------------------------------------------------------------------
 // PluginHandler::findPluginByExtension