[EMAIL PROTECTED] wrote:
On Jun 17, 8:02 am, bvdp <[EMAIL PROTECTED]> wrote:

Thanks. That was easy :)

The change to the _ast version is left as an exercise to the reader ;)
And I have absolutely no idea on how to do this. I can't even find the
_ast import file on my system. I'm assuming that the _ast definitions
are buried in the C part of python, but that is just a silly guess.

Bob.

If you just need numeric expressions with a small number of functions,
I would suggest checking the expression string first with a simple
regular expression, then using the standard eval() to evaluate the
result.  This blocks the attacks mentioned above, and is simple to
implement.  This will not work if you want to allow string values in
expressions though.

import re
def safe_eval( expr, safe_cmds=[] ):
        toks = re.split( r'([a-zA-Z_\.]+|.)', expr )
        bad = [t for t in toks if len(t)>1 and t not in safe_cmds]
        if not bad:
                return eval( expr )


Yes, this appears to be about as good (better?) an idea as any. Certainly beats writing my own recursive decent parser for this :)

And it is not dependent on python versions. Cool.

I've run a few tests with your code and it appears to work just fine. Just a matter of populating the save_cmds[] array and putting in some error traps. Piece of cake. And should be fast as well.

Thanks!!!

Bob.
--
http://mail.python.org/mailman/listinfo/python-list

Reply via email to