While initializing the pepper plugin registry in the renderer we load each plugin and attempt to ...

webkit/plugins/ppapi/plugin_module.cc

Index: webkit/plugins/ppapi/plugin_module.cc
===================================================================
--- webkit/plugins/ppapi/plugin_module.cc	(revision 91384)
+++ webkit/plugins/ppapi/plugin_module.cc	(working copy)
@@ -458,6 +458,7 @@
 
   if (!LoadEntryPointsFromLibrary(library, &entry_points_) ||
       !InitializeModule()) {
+    memset(&entry_points_, 0, sizeof(entry_points_));

[brettw, 2011/07/06 18:27:54]

Can we do this in the constructor instead?

[ananta, 2011/07/06 18:41:53]

I moved the memset to the constructor. However the problem is not that this
structure is uninitialized. Basically we load the entry points and then try to
initialize the plugin. If the initialize fails we try to uninitialize it which
crashes.

Changed the code around to only initialize the entry points member if the plugin
initialization succeeds.

     base::UnloadNativeLibrary(library);
     return false;
   }