luzer: fix type in consume_number()

The implementation of FDP methods `consume_number()` and
`consume_numbers()` are used type `double` for arguments of
aforementioned functions. The Lua type `lua_Number` can be not a
`double` and it is defined compile-time. The patch replaces
`double` with `lua_Number` to avoid overflows.

Follows up #29

Author: ligurio

CHANGELOG.md

@@ -22,6 +22,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Disable coverage instrumentation of internal functions (#11).
 - Add missed newlines to messages.
 - Rename `_VERSION` to a `_LUZER_VERSION`.
+- Use `lua_Number` in FDP methods `consume_number()` and
+  `consume_numbers()` instead `double`.
 
 ### Fixed
 

luzer/fuzzed_data_provider.cc

@@ -110,12 +110,12 @@ luaL_consume_number(lua_State *L)
 {
 	lua_userdata_t *lfdp;
 	lfdp = (lua_userdata_t *)luaL_checkudata(L, 1, FDP_LUA_UDATA_NAME);
-	double min = luaL_checknumber(L, 2);
-	double max = luaL_checknumber(L, 3);
+	lua_Number min = luaL_checknumber(L, 2);
+	lua_Number max = luaL_checknumber(L, 3);
 	if (min > max)
 		luaL_error(L, "min must be less than or equal to max");
 
-	auto number = lfdp->fdp->ConsumeFloatingPointInRange(min, max);
+	lua_Number number = lfdp->fdp->ConsumeFloatingPointInRange(min, max);
 	lua_pushnumber(L, number);
 
 	return 1;
@@ -127,15 +127,15 @@ luaL_consume_numbers(lua_State *L)
 {
 	lua_userdata_t *lfdp;
 	lfdp = (lua_userdata_t *)luaL_checkudata(L, 1, FDP_LUA_UDATA_NAME);
-	int count = luaL_checkinteger(L, 2);
-	double min = luaL_checkinteger(L, 3);
-	double max = luaL_checkinteger(L, 4);
+	lua_Integer count = luaL_checkinteger(L, 2);
+	lua_Number min = luaL_checkinteger(L, 3);
+	lua_Number max = luaL_checkinteger(L, 4);
 	if (min > max)
 		luaL_error(L, "min must be less than or equal to max");
 
 	lua_newtable(L);
-	for (int i = 1; i <= count; i++) {
-		auto number = lfdp->fdp->ConsumeFloatingPointInRange(min, max);
+	for (lua_Integer i = 1; i <= count; i++) {
+		lua_Number number = lfdp->fdp->ConsumeFloatingPointInRange(min, max);
 		lua_pushnumber(L, i);
 		lua_pushnumber(L, number);
 		lua_settable(L, -3);

luzer/tests/test_unit.lua

@@ -131,6 +131,9 @@ ok, err = pcall(fdp.consume_number)
 assert(ok == false)
 assert(err ~= nil)
 
+local num = fdp:consume_number(1, MAX_INT)
+assert(type(num) == "number")
+
 -- luzer.FuzzedDataProvider.consume_numbers()
 fdp = luzer.FuzzedDataProvider("ABCDEF")
 assert(type(fdp.consume_numbers) == "function")
@@ -160,7 +163,7 @@ assert(err ~= nil)
 
 local max_int = fdp:consume_integer(MAX_INT, MAX_INT)
 assert(type(max_int) == "number")
--- FIXME: assert(max_int == MAX_INT, max_int)
+assert(max_int == MAX_INT, max_int)
 
 local min_int = fdp:consume_integer(MIN_INT, MIN_INT)
 assert(type(min_int) == "number")