Skip to content

fix(actionmanager): Improve validation of Sneak Attack placement legality #2533

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
xezon merged 6 commits into from
Apr 6, 2026
Merged

fix(actionmanager): Improve validation of Sneak Attack placement legality #2533

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
581ad51
fix(logic): Use const builderObject in BuildAssistant::isLocationLega…
stephanmeesters Apr 4, 2026
fc1aa4e
fix(logic): Improve validation of building placement for Sneak Attack
stephanmeesters Apr 1, 2026
348f7ca
Replace elif with else
stephanmeesters Apr 4, 2026
9d50bc0
Process review comments
stephanmeesters Apr 4, 2026
5e8717c
Process review comments 2
stephanmeesters Apr 6, 2026
8e5fd24
Replicate to Generals
stephanmeesters Apr 6, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions GeneralsMD/Code/GameEngine/Include/Common/BuildAssistant.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -151,14 +151,14 @@ class BuildAssistant : public SubsystemInterface
const ThingTemplate *build,
Real angle, // angle to construct 'build' at
UnsignedInt options, // use LocationLegalToBuildOptions
Object *builderObject,
const Object *builderObject,
Player *player);

/// query if we can build at this location
virtual LegalBuildCode isLocationClearOfObjects( const Coord3D *worldPos,
const ThingTemplate *build,
Real angle, // angle to construct 'build' a
Object *builderObject,
const Object *builderObject,
UnsignedInt options,
Player *thePlayer);

Expand Down
33 changes: 31 additions & 2 deletions GeneralsMD/Code/GameEngine/Source/Common/RTS/ActionManager.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,7 @@
#include "PreRTS.h" // This must go first in EVERY cpp file in the GameEngine

#include "Common/ActionManager.h"
#include "Common/BuildAssistant.h"
#include "Common/GlobalData.h"
#include "Common/Player.h"
#include "Common/PlayerList.h"
Expand Down Expand Up @@ -1506,7 +1507,7 @@ Bool ActionManager::canDoSpecialPowerAtLocation( const Object *obj, const Coord3
}
}

// Last check is shroudedness, if it is cared about
// Second check is shroudedness, if it is cared about
switch( spTemplate->getSpecialPowerType() )
{
case SPECIAL_DAISY_CUTTER:
Expand Down Expand Up @@ -1552,7 +1553,6 @@ Bool ActionManager::canDoSpecialPowerAtLocation( const Object *obj, const Coord3
case SUPW_SPECIAL_PARTICLE_UPLINK_CANNON:
case LAZR_SPECIAL_PARTICLE_UPLINK_CANNON:
case SPECIAL_CLEANUP_AREA:
case SPECIAL_SNEAK_ATTACK:
case SPECIAL_BATTLESHIP_BOMBARDMENT:
//Don't allow "damaging" special powers in shrouded areas, but Fogged are okay.
return ThePartitionManager->getShroudStatusForPlayer( obj->getControllingPlayer()->getPlayerIndex(), loc ) != CELLSHROUD_SHROUDED;
Expand Down Expand Up @@ -1586,6 +1586,35 @@ Bool ActionManager::canDoSpecialPowerAtLocation( const Object *obj, const Coord3
case SPECIAL_CHANGE_BATTLE_PLANS:
return false;
}

// TheSuperHackers @fix stephanmeesters 04/04/2026 Some special powers can spawn a building. To avoid cheating
Comment thread
stephanmeesters marked this conversation as resolved.
Outdated
// we must verify that it is legal to place this building.
Comment thread
stephanmeesters marked this conversation as resolved.
Outdated
switch( spTemplate->getSpecialPowerType() )
{
case SPECIAL_SNEAK_ATTACK:
{
#if RETAIL_COMPATIBLE_CRC
return ThePartitionManager->getShroudStatusForPlayer( obj->getControllingPlayer()->getPlayerIndex(), loc ) != CELLSHROUD_SHROUDED;
#elif
Comment thread
greptile-apps[bot] marked this conversation as resolved.
Outdated
const ThingTemplate* referenceThing = mod->getReferenceThingTemplate();
DEBUG_ASSERTCRASH(referenceThing, ("canDoSpecialPowerAtLocation: SpecialPowerTemplate has a null ThingTemplate") );
Comment thread
Skyaero42 marked this conversation as resolved.
Outdated
if (!referenceThing)
return FALSE;

return TheBuildAssistant->isLocationLegalToBuild(loc,
Comment thread
xezon marked this conversation as resolved.
Outdated
referenceThing,
referenceThing->getPlacementViewAngle(),
BuildAssistant::USE_QUICK_PATHFIND |
BuildAssistant::TERRAIN_RESTRICTIONS |
BuildAssistant::CLEAR_PATH |
BuildAssistant::NO_OBJECT_OVERLAP |
BuildAssistant::SHROUD_REVEALED |
BuildAssistant::IGNORE_STEALTHED,
obj,
nullptr) == LBC_OK;
#endif
}
}
}
return false;
}
Expand Down
6 changes: 3 additions & 3 deletions GeneralsMD/Code/GameEngine/Source/Common/System/BuildAssistant.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -656,7 +656,7 @@ void BuildAssistant::iterateFootprint( const ThingTemplate *build,
LegalBuildCode BuildAssistant::isLocationClearOfObjects( const Coord3D *worldPos,
const ThingTemplate *build,
Real angle,
Object *builderObject,
const Object *builderObject,
UnsignedInt options,
Player *thePlayer)
{
Expand Down Expand Up @@ -915,7 +915,7 @@ LegalBuildCode BuildAssistant::isLocationLegalToBuild( const Coord3D *worldPos,
const ThingTemplate *build,
Real angle,
UnsignedInt options,
Object *builderObject,
const Object *builderObject,
Player *player)
{

Expand Down Expand Up @@ -1002,7 +1002,7 @@ LegalBuildCode BuildAssistant::isLocationLegalToBuild( const Coord3D *worldPos,
// if clear path is requested check to see if the builder object can get there (unless it's a structure)
if( BitIsSet( options, CLEAR_PATH ) && builderObject && !builderObject->isKindOf( KINDOF_IMMOBILE ) )
{
AIUpdateInterface *ai = builderObject->getAIUpdateInterface();
const AIUpdateInterface *ai = builderObject->getAIUpdateInterface();

//
// if there is no AI interface for this object, it cannot possible pass a clear path
Expand Down
Loading