ABP/Android Fix crash on unescaped uri

jni/jsEngine.cpp

 /*
  * This file is part of the Adblock Plus,
  * Copyright (C) 2006-2012 Eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 146 matching lines @@
 {
   D(D_WARN, "nativeExecute()");
   v8::HandleScope handle_scope;
 
   v8::Persistent<v8::Context> context((v8::Context *) pContext);
   v8::Context::Scope context_scope(context);
 
   const std::string script = getString(pEnv, pScript);
 
   v8::Handle<v8::String> source = v8::String::New(script.c_str(), script.size());
-  v8::Handle<v8::Script> compiledScript = v8::Script::Compile(source);
+  v8::Handle<v8::Script> compiledScript;
+  {
+    v8::TryCatch try_catch;

[Felix Dahlke, 2013/03/06 15:06:35]

You could just use the same TryCatch for both Compile and Run. Anyway, LGTM.

+    compiledScript = v8::Script::Compile(source);
+    if (try_catch.HasCaught())
+    {
+      reportException(&try_catch);
+      return NULL;
+    }
+  }
   {
     v8::TryCatch try_catch;
     v8::Handle<v8::Value> result = compiledScript->Run();
     if (try_catch.HasCaught())
     {
       reportException(&try_catch);
       return NULL;
     }
     else
     {
@@ skipping 93 matching lines @@
   long r = 0;
   while (r == 0)
   {
     v8::TryCatch try_catch;
     r = RunNextCallback(context);
     if (try_catch.HasCaught())
       reportException(&try_catch);
   }
   return (jlong) r;
 }