Make ExternalizeStringExtension free its data on exit to please

src/execution.cc

 // Copyright 2006-2008 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 779 matching lines @@
   }
   if (string->IsAsciiRepresentation() && !force_two_byte) {
     char* data = new char[string->length()];
     String::WriteToFlat(*string, data, 0, string->length());
     SimpleAsciiStringResource* resource = new SimpleAsciiStringResource(
         data, string->length());
     result = string->MakeExternal(resource);
     if (result && !string->IsSymbol()) {
       i::ExternalStringTable::AddString(*string);
     }
+    if (!result) delete resource;
   } else {
     uc16* data = new uc16[string->length()];
     String::WriteToFlat(*string, data, 0, string->length());
     SimpleTwoByteStringResource* resource = new SimpleTwoByteStringResource(
         data, string->length());
     result = string->MakeExternal(resource);
     if (result && !string->IsSymbol()) {
       i::ExternalStringTable::AddString(*string);
     }
+    if (!result) delete resource;
   }
   if (!result) {
     return v8::ThrowException(v8::String::New("externalizeString() failed."));
   }
   return v8::Undefined();
 }
 
 
 v8::Handle<v8::Value> ExternalizeStringExtension::IsAscii(
     const v8::Arguments& args) {
   if (args.Length() != 1 || !args[0]->IsString()) {
     return v8::ThrowException(v8::String::New(
         "isAsciiString() requires a single string argument."));
   }
   return Utils::OpenHandle(*args[0].As<v8::String>())->IsAsciiRepresentation() ?
       v8::True() : v8::False();
 }
 
 
 static ExternalizeStringExtension externalize_extension;
 static v8::DeclareExtension externalize_extension_declaration(
     &externalize_extension);
 
 } }  // namespace v8::internal