Fix several ownership/lifetime bugs in argument wire types

3 files changed, 10 insertions, 28 deletions

diff --git a/system/include/emscripten/bind.h b/system/include/emscripten/bind.h
index 2de14fab..69d1f08c 100644
--- a/system/include/emscripten/bind.h
+++ b/system/include/emscripten/bind.h
@@ -223,7 +223,7 @@ namespace emscripten {
         }
 
         template<typename PointerType>
-        typename PointerType::element_type* get_pointee(PointerType ptr) {
+        typename PointerType::element_type* get_pointee(const PointerType& ptr) {
             // TODO: replace with general pointer traits implementation
             return ptr.get();
         }
diff --git a/system/include/emscripten/val.h b/system/include/emscripten/val.h
index ada89638..8369caf7 100644
--- a/system/include/emscripten/val.h
+++ b/system/include/emscripten/val.h
@@ -194,7 +194,7 @@ namespace emscripten {
                 return v.handle;
             }
             static val fromWireType(WireType v) {
-                return val(v);
+                return val::take_ownership(v);
             }
         };
     }
diff --git a/system/include/emscripten/wire.h b/system/include/emscripten/wire.h
index b618be83..ceabd280 100644
--- a/system/include/emscripten/wire.h
+++ b/system/include/emscripten/wire.h
@@ -130,7 +130,7 @@ namespace emscripten {
         template<>
         struct BindingType<std::string> {
             typedef char* WireType;
-            static WireType toWireType(std::string v) {
+            static WireType toWireType(const std::string& v) {
                 return strdup(v.c_str());
             }
             static std::string fromWireType(char* v) {
@@ -138,14 +138,14 @@ namespace emscripten {
             }
         };
 
-        template<>
-        struct BindingType<const std::string&> {
-            typedef char* WireType;
-            static WireType toWireType(std::string v) {
-                return strdup(v.c_str());
+        template<typename T>
+        struct BindingType<const T&> {
+            typedef typename BindingType<T>::WireType WireType;
+            static WireType toWireType(const T& v) {
+                return BindingType<T>::toWireType(v);
             }
-            static std::string fromWireType(char* v) {
-                return std::string(v);
+            static T fromWireType(WireType wt) {
+                return BindingType<T>::fromWireType(wt);
             }
         };
 
@@ -210,24 +210,6 @@ namespace emscripten {
         };
 
         template<typename T>
-        struct GenericBindingType<std::shared_ptr<T>> {
-            typedef typename std::shared_ptr<T> ActualT;
-            typedef ActualT* WireType;
-
-            static WireType toWireType(std::shared_ptr<T> p) {
-                return new std::shared_ptr<T>(p);
-            }
-
-            static std::shared_ptr<T> fromWireType(WireType p) {
-                return *p;
-            }
-
-            static void destroy(WireType p) {
-                delete p;
-            }
-        };
-
-        template<typename T>
         struct WireDeleter {
             typedef typename BindingType<T>::WireType WireType;