tests with GMP objects instances too

Author: devnexen

ext/gmp/gmp.c

@@ -1588,10 +1588,17 @@ ZEND_FUNCTION(gmp_invert)
 	if (Z_TYPE_P(b_arg) == IS_LONG && Z_LVAL_P(b_arg) == 0) {
 		zend_throw_exception_ex(zend_ce_division_by_zero_error, 0, "Division by zero");
 		RETURN_THROWS();
-	}
+	} else {
+		FETCH_GMP_ZVAL(gmpnum_a, a_arg, temp_a, 1);
+		FETCH_GMP_ZVAL_DEP(gmpnum_b, b_arg, temp_b, temp_a, 2);
 
-	FETCH_GMP_ZVAL(gmpnum_a, a_arg, temp_a, 1);
-	FETCH_GMP_ZVAL_DEP(gmpnum_b, b_arg, temp_b, temp_a, 2);
+		if (!mpz_cmp_ui(gmpnum_b, 0)) {
+			zend_throw_exception_ex(zend_ce_division_by_zero_error, 0, "Division by zero");
+			FREE_GMP_TEMP(temp_a);
+			FREE_GMP_TEMP(temp_b);
+			RETURN_THROWS();
+		}
+	}
 
 	INIT_GMP_RETVAL(gmpnum_result);
 	res = mpz_invert(gmpnum_result, gmpnum_a, gmpnum_b);

ext/gmp/tests/gmp_invert.phpt

@@ -15,6 +15,13 @@ try {
     echo $e->getMessage() . \PHP_EOL;
 }
 
+try {
+    $zero = new GMP(0);
+    var_dump(gmp_invert(5, $zero));
+} catch (\DivisionByZeroError $e) {
+    echo $e->getMessage() . \PHP_EOL;
+}
+
 var_dump(gmp_strval(gmp_invert(0,28347)));
 var_dump(gmp_strval(gmp_invert(-12,456456)));
 var_dump(gmp_strval(gmp_invert(234234,-435345)));
@@ -48,6 +55,7 @@ string(7) "2293131"
 string(1) "0"
 string(4) "5827"
 Division by zero
+Division by zero
 string(1) "0"
 string(1) "0"
 string(1) "0"