BugFix - Escapable scope was not defined in GetValue

Bigous · Bigous · commit 45eb8dedcbce · 2015-03-10T11:43:23.000-03:00
Signed-off-by: Richard Natal &lt;bigous@gmail.com&gt;
diff --git a/src/njs/src/njsConnection.cpp b/src/njs/src/njsConnection.cpp
@@ -1038,7 +1038,7 @@ v8::Handle<v8::Value> Connection::GetRows (eBaton* executeBaton)
 v8::Handle<v8::Value> Connection::GetValue ( short ind, unsigned short type, void* val,
                                      DPI_BUFLEN_TYPE len )
 {
-  NanScope();
+  NanEscapableScope();
   Handle<Value> value;
   Local<Date> date;
   if(ind != -1)
@@ -1067,7 +1067,7 @@ v8::Handle<v8::Value> Connection::GetValue ( short ind, unsigned short type, voi
   {
     value = NanNull();
   }
-  return value;
+  return NanEscapeScope(value);
 }
 
 /*****************************************************************************/

Original file line number	Diff line number	Diff line change
`@@ -1038,7 +1038,7 @@ v8::Handle<v8::Value> Connection::GetRows (eBaton* executeBaton)`
`1038`	`1038`	`v8::Handle<v8::Value> Connection::GetValue ( short ind, unsigned short type, void* val,`
`1039`	`1039`	`DPI_BUFLEN_TYPE len )`
`1040`	`1040`	`{`
`1041`		`- NanScope();`
	`1041`	`+ NanEscapableScope();`
`1042`	`1042`	`Handle<Value> value;`
`1043`	`1043`	`Local<Date> date;`
`1044`	`1044`	`if(ind != -1)`
`@@ -1067,7 +1067,7 @@ v8::Handle<v8::Value> Connection::GetValue ( short ind, unsigned short type, voi`
`1067`	`1067`	`{`
`1068`	`1068`	`value = NanNull();`
`1069`	`1069`	`}`
`1070`		`- return value;`
	`1070`	`+ return NanEscapeScope(value);`
`1071`	`1071`	`}`
`1072`	`1072`
`1073`	`1073`	`/*****************************************************************************/`