From fc9adb62fbfa8d2f81f7224afea43557a4fff409 Mon Sep 17 00:00:00 2001 From: Mark Dickinson Date: Sat, 6 Oct 2012 18:50:02 +0100 Subject: [PATCH] Issue #16096: Fix signed overflow in Objects/longobject.c. Thanks Serhiy Storchaka. --- Objects/longobject.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/Objects/longobject.c b/Objects/longobject.c index 73413dd5afb..4cc080f2b5d 100644 --- a/Objects/longobject.c +++ b/Objects/longobject.c @@ -668,10 +668,9 @@ _PyLong_NumBits(PyObject *vv) assert(ndigits == 0 || v->ob_digit[ndigits - 1] != 0); if (ndigits > 0) { digit msd = v->ob_digit[ndigits - 1]; - - result = (ndigits - 1) * PyLong_SHIFT; - if (result / PyLong_SHIFT != (size_t)(ndigits - 1)) + if ((size_t)(ndigits - 1) > PY_SIZE_MAX / (size_t)PyLong_SHIFT) goto Overflow; + result = (size_t)(ndigits - 1) * (size_t)PyLong_SHIFT; do { ++result; if (result == 0)