Fix array index out of bounds due to malformed plugin message (PR539)

Failed calls to getString and getMember on JSObjects should not produce malformed
result strings. "null" is appended to result rather than empty string.

* plugin/icedteanp/IcedTeaPluginRequestProcessor.cc: (_getMember,
_getString) append "null" to result when call is unsuccessful
* tests/reproducers/simple/JSObjectWithoutToString/resources/JSObjectWithoutToString.html:
new test to ensure failed calls to getMember and getString on JSObject do
not produce malformed results
* tests/reproducers/simple/JSObjectWithoutToString/resources/JSObjectWithoutToString.js:
same
* tests/reproducers/simple/JSObjectWithoutToString/srcs/JSObjectWithoutToString.java:
same
* tests/reproducers/simple/JSObjectWithoutToString/testcases/JSObjectWithoutToStringTest.java:
same

1 files changed, 15 insertions, 0 deletions

diff --git a/ChangeLog b/ChangeLog
index 2d48963..d5c9dd8 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,18 @@
+2013-10-24  Andrew Azores  <[email protected]>
+
+	Fix array index out of bounds due to malformed plugin message (PR539)
+	* plugin/icedteanp/IcedTeaPluginRequestProcessor.cc: (_getMember,
+	_getString) append "null" to result when call is unsuccessful
+	* tests/reproducers/simple/JSObjectWithoutToString/resources/JSObjectWithoutToString.html:
+	new test to ensure failed calls to getMember and getString on JSObject do
+	not produce malformed results
+	* tests/reproducers/simple/JSObjectWithoutToString/resources/JSObjectWithoutToString.js:
+	same
+	* tests/reproducers/simple/JSObjectWithoutToString/srcs/JSObjectWithoutToString.java:
+	same
+	* tests/reproducers/simple/JSObjectWithoutToString/testcases/JSObjectWithoutToStringTest.java:
+	same
+
 2013-10-23  Jiri Vanek  <[email protected]>
 
 	C-part of plugin is now also trying to follow XDG