Add value range check for seed

src/mmh3/mmh3module.c

@@ -51,10 +51,18 @@ typedef unsigned __int64 uint64_t;
                          Py_TYPE(args[1])->tp_name);                        \
             return NULL;                                                    \
         }                                                                   \
-        seed = (uint32_t)PyLong_AsUnsignedLong(args[1]);                    \
-        if (seed == (unsigned long)-1 && PyErr_Occurred()) {                \
+        const unsigned long seed_tmp = PyLong_AsUnsignedLong(args[1]);      \
+        if (seed_tmp == -1 && PyErr_Occurred()) {                           \
+            if (PyErr_ExceptionMatches(PyExc_OverflowError)) {              \
+                PyErr_SetString(PyExc_ValueError, "seed is out of range");  \
+                return NULL;                                                \
+            }                                                               \
+        }                                                                   \
+        if (seed_tmp > 0xFFFFFFFF) {                                        \
+            PyErr_SetString(PyExc_ValueError, "seed is out of range");      \
             return NULL;                                                    \
         }                                                                   \
+        seed = (uint32_t)seed_tmp;                                          \
     }
 
 //-----------------------------------------------------------------------------

tests/test_invalid_inputs.py

@@ -0,0 +1,24 @@
+# pylint: disable=missing-module-docstring, missing-function-docstring
+# pylint: disable=no-value-for-parameter, too-many-function-args
+import mmh3
+import pytest
+
+
+def test_mmh3_32_digest_raises_typeerror() -> None:
+    with pytest.raises(TypeError):
+        mmh3.mmh3_32_digest()
+    with pytest.raises(TypeError):
+        mmh3.mmh3_32_digest(b"hello, world", 42, 1234)
+    with pytest.raises(TypeError):
+        mmh3.mmh3_32_digest("hello, world")
+    with pytest.raises(TypeError):
+        mmh3.mmh3_32_digest(b"hello, world", "42")
+    with pytest.raises(TypeError):
+        mmh3.mmh3_32_digest([1, 2, 3], 42)
+
+
+def test_mmh3_32_digest_raises_valueerror() -> None:
+    with pytest.raises(ValueError):
+        mmh3.mmh3_32_digest(b"hello, world", -1)
+    with pytest.raises(ValueError):
+        mmh3.mmh3_32_digest(b"hello, world", 2**32)

tox.ini

@@ -8,7 +8,7 @@ description = run unit tests
 commands_pre =
     pip install ".[test]"
 commands =
-    pytest
+    pytest {posargs}
 
 [testenv:lint]
 description = run linters with formatting