mirror of
https://github.com/minetest/minetest.git
synced 2024-11-30 03:23:45 +01:00
Mapgen V7: Fix out-of-bounds array access
This commit is contained in:
parent
06266abd6e
commit
65874c25db
@ -427,19 +427,19 @@ void MapgenV7::generateMountainTerrain() {
|
|||||||
return;
|
return;
|
||||||
|
|
||||||
MapNode n_stone(c_stone);
|
MapNode n_stone(c_stone);
|
||||||
u32 index = 0;
|
u32 j = 0;
|
||||||
|
|
||||||
for (s16 z = node_min.Z; z <= node_max.Z; z++)
|
for (s16 z = node_min.Z; z <= node_max.Z; z++)
|
||||||
for (s16 y = node_min.Y; y <= node_max.Y; y++) {
|
for (s16 y = node_min.Y; y <= node_max.Y; y++) {
|
||||||
u32 vi = vm->m_area.index(node_min.X, y, z);
|
u32 vi = vm->m_area.index(node_min.X, y, z);
|
||||||
for (s16 x = node_min.X; x <= node_max.X; x++) {
|
for (s16 x = node_min.X; x <= node_max.X; x++) {
|
||||||
int j = (z - node_min.Z) * csize.X + (x - node_min.X);
|
int index = (z - node_min.Z) * csize.X + (x - node_min.X);
|
||||||
|
|
||||||
if (getMountainTerrainFromMap(index, j, y))
|
if (getMountainTerrainFromMap(j, index, y))
|
||||||
vm->m_data[vi] = n_stone;
|
vm->m_data[vi] = n_stone;
|
||||||
|
|
||||||
vi++;
|
vi++;
|
||||||
index++;
|
j++;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
@ -517,7 +517,9 @@ void MapgenV7::generateBiomes() {
|
|||||||
// boundary, but the chunk above has not been generated yet
|
// boundary, but the chunk above has not been generated yet
|
||||||
if (y == node_max.Y && c_above == CONTENT_IGNORE &&
|
if (y == node_max.Y && c_above == CONTENT_IGNORE &&
|
||||||
y == heightmap[index] && c == c_stone) {
|
y == heightmap[index] && c == c_stone) {
|
||||||
int j = z * zstride + y * ystride + x;
|
int j = (z - node_min.Z) * zstride +
|
||||||
|
(y - node_min.Y) * ystride +
|
||||||
|
(x - node_min.X);
|
||||||
have_air = !getMountainTerrainFromMap(j, index, y);
|
have_air = !getMountainTerrainFromMap(j, index, y);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user