Bugs item #1887838, was opened at 2008-02-06 13:19
Message generated for change (Comment added) made by lebert
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=111005&aid=1887838&group_id=11005
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Program
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Bert Wesarg (lebert)
Assigned to: Nobody/Anonymous (nobody)
Summary: fix possible stack overflow in interpreter
Initial Comment:
the check for stack overflow in pushSymVal() should be done before writing to
the stack
----------------------------------------------------------------------
>Comment By: Bert Wesarg (lebert)
Date: 2008-02-07 15:43
Message:
Logged In: YES
user_id=122956
Originator: YES
follow-up patch updated: make it compile and remove variable shadowing
File Added: interpret-macro-cleanup.patch
----------------------------------------------------------------------
Comment By: Bert Wesarg (lebert)
Date: 2008-02-07 15:23
Message:
Logged In: YES
user_id=122956
Originator: YES
follow-up patch: refactor PUSH/POP/PEEK macros
use 'do {} while(0)' for these macros, that means add a ';' after all
calls
because this touches almost all functions, I haven't done code style fixes
File Added: interpret-macro-cleanup.patch
----------------------------------------------------------------------
Comment By: Bert Wesarg (lebert)
Date: 2008-02-07 15:14
Message:
Logged In: YES
user_id=122956
Originator: YES
updated patch: move variable declaration in front of code
File Added: fix-stack-overflow-check.patch
----------------------------------------------------------------------
Comment By: Tony Balinski (ajbj)
Date: 2008-02-06 22:30
Message:
Logged In: YES
user_id=618141
Originator: NO
Nicely done.
----------------------------------------------------------------------
Comment By: Bert Wesarg (lebert)
Date: 2008-02-06 14:52
Message:
Logged In: YES
user_id=122956
Originator: YES
there where some more handcrafted stack pops/pushs including one more with
the check after the assigment. this patch converts these to use the
POP/PUSH macros and do some style clean ups.
File Added: fix-stack-overflow-check.patch
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=111005&aid=1887838&group_id=11005
--
NEdit Develop mailing list - [email protected]
http://www.nedit.org/mailman/listinfo/develop
