Stack Exhaustion (ecma_proxy_object_get, ecma_proxy_object_set) #3785

Activity

1. akosthekiss commented on May 23, 2020

   

   akosthekiss

   on May 23, 2020

   Member

   More actions

   Could you please try whether you get SO even if you set a limit for the stack? (Note: You can use --stack-limit option when building the engine to limit the maximum amount of stack that the engine can use.)
2. nszetei commented on May 23, 2020

   

   nszetei

   on May 23, 2020

   Author

   More actions

   Could you please try whether you get SO even if you set a limit for the stack? (Note: You can use --stack-limit option when building the engine to limit the maximum amount of stack that the engine can use.)

   Yes. Unlike #3783 here I got SO in both cases (e.g. with --stack-limit=10).

   👍React with 👍1Reacted by Robert Fancsik
3. nszetei commented on May 23, 2020

   

   nszetei

   on May 23, 2020

   Author

   More actions

   Just to have it grouped together, a PoC for ecma_proxy_object_has:

   function main() {
   var v1 = [13.37,13.37,13.37,13.37];
   var v4 = {isExtensible:Infinity};
   var v6 = new Proxy(WeakMap,v4);
   v6.__proto__ = v1;
   v1.__proto__ = v6;
   with (v1) {
       valueOf = 0;
   }
   }
   main();
   
4. dbatyai

   self-assigned this

   on May 25, 2020
5. dbatyai

   added

   bugUndesired behaviourUndesired behaviour

   on May 25, 2020
6. dbatyai

   added 3 commits that reference this issue on May 25, 2020

   Add stack limit check to proxy operations

   ...

   This issue will close once commit 012b81c is merged into the 'master' branch.

   012b81c

   Add stack limit check to proxy operations

   ...

   This issue will close once commit e2e4818 is merged into the 'master' branch.

   e2e4818

   Add stack limit check to proxy operations

   ...

   This issue will close once commit a096da7 is merged into the 'master' branch.

   a096da7
7. zherczeg

   closed this as completedin #3796on May 26, 2020
8. zherczeg

   added a commit that references this issue on May 26, 2020

   Add stack limit check to proxy operations (#3796)

   ...

   This issue will close once commit 15629e8 is merged into the 'master' branch.

   15629e8