From 2b575418b6c0b97966d77fc1b093c5ed34d92670 Mon Sep 17 00:00:00 2001 From: Damien George Date: Tue, 19 Feb 2019 23:44:01 +1100 Subject: [PATCH] py/qstr: Evaluate find_qstr only once then pass to Q_GET_HASH macro. Q_GET_HASH may evaluate its argument more than once. --- py/qstr.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/py/qstr.c b/py/qstr.c index a06b84f153..e6f86401a0 100644 --- a/py/qstr.c +++ b/py/qstr.c @@ -251,7 +251,8 @@ qstr qstr_from_strn(const char *str, size_t len) { } mp_uint_t qstr_hash(qstr q) { - return Q_GET_HASH(find_qstr(q)); + const byte *qd = find_qstr(q); + return Q_GET_HASH(qd); } size_t qstr_len(qstr q) {