Skip to content
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

Add a safeguard to map command #529

Merged
merged 9 commits into from
Oct 2, 2023
Merged

Add a safeguard to map command #529

Changes from 5 commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
ad9738d
add a safeguard to map command
catornot Aug 2, 2023
cc61f34
add RefreshMapList into the command
catornot Aug 2, 2023
52823a5
reduce nesting and improve the warn msg
catornot Aug 3, 2023
705cab8
add a warning for missing map name
catornot Aug 4, 2023
8dfc703
Merge branch 'main' into map_safeguard
Alystrasz Aug 6, 2023
aa1c9e8
reconstruct map concommand
catornot Aug 9, 2023
ffae7c5
Merge branch 'map_safeguard' of https://github.com/catornot/Northstar…
catornot Aug 9, 2023
0757e7a
use nullptr instead of std::nullptr_t
catornot Aug 27, 2023
dfc26d1
Merge branch 'main' into map_safeguard
catornot Oct 1, 2023
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
24 changes: 24 additions & 0 deletions NorthstarDLL/util/printmaps.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,8 @@ struct MapVPKInfo
// our current list of maps in the game
std::vector<MapVPKInfo> vMapList;

FnCommandCallback_t OriginalMapCommand = NULL;

void RefreshMapList()
{
// Only update the maps list every 10 seconds max to we avoid constantly reading fs
Expand Down Expand Up @@ -182,10 +184,32 @@ void ConCommand_maps(const CCommand& args)
spdlog::info("({}) {}", PrintMapSource.at(map.source), map.name);
}

void ConCommand_map(const CCommand& args)
{
RefreshMapList();

if (args.ArgC() > 1 &&
std::find_if(vMapList.begin(), vMapList.end(), [&](MapVPKInfo map) -> bool { return map.name == args.Arg(1); }) == vMapList.end())
{
spdlog::warn("Map load failed: {} not found or invalid", args.Arg(1));
return;
}
else
{
spdlog::warn("Map load failed: no map name provided");
}

OriginalMapCommand(args);
}

void InitialiseMapsPrint()
{
AUTOHOOK_DISPATCH()

ConCommand* mapsCommand = R2::g_pCVar->FindCommand("maps");
mapsCommand->m_pCommandCallback = ConCommand_maps;

ConCommand* mapCommand = R2::g_pCVar->FindCommand("map");
OriginalMapCommand = mapCommand->m_pCommandCallback;
mapCommand->m_pCommandCallback = ConCommand_map;
}