656324 - functions wrapped with sync() cause StackOverflowError when setting properties

Status: RESOLVED INACTIVE

(Rhino Graveyard :: Core, defect)

Description

[John Spackman]

User-Agent:       Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10_6_7; en-US) AppleWebKit/534.16 (KHTML, like Gecko) Chrome/10.0.648.205 Safari/534.16
Build Identifier: Rhino 1.7 release 3 PRERELEASE 2011 03 17 

If you use sync() to wrap a function and then add properties to the returned function object, setting the property causes a StackOverflowError.

Reproducible: Always

Steps to Reproduce:
var f = sync( function() {} );
f.displayName = 'hello';

Actual Results:  
java.lang.StackOverflowError
	at org.mozilla.javascript.ScriptableObject.getSlot(ScriptableObject.java:2577)
	at org.mozilla.javascript.ScriptableObject.putImpl(ScriptableObject.java:2527)
	at org.mozilla.javascript.ScriptableObject.put(ScriptableObject.java:463)
	at org.mozilla.javascript.IdScriptableObject.put(IdScriptableObject.java:424)
	at org.mozilla.javascript.Delegator.put(Delegator.java:149)
	at org.mozilla.javascript.ScriptableObject.put(ScriptableObject.java:467)
	at org.mozilla.javascript.IdScriptableObject.put(IdScriptableObject.java:424)
	at org.mozilla.javascript.Delegator.put(Delegator.java:149)
	at org.mozilla.javascript.ScriptableObject.put(ScriptableObject.java:467)
	at org.mozilla.javascript.IdScriptableObject.put(IdScriptableObject.java:424)
	at org.mozilla.javascript.Delegator.put(Delegator.java:149)
	at org.mozilla.javascript.ScriptableObject.put(ScriptableObject.java:467)
... etc ...

Expected Results:  
no error

Comment 1

[Sylvestre Ledru [:Sylvestre]]

Closing. Bug management is now done here:
https://github.com/mozilla/rhino