mirror of
https://git.code.sf.net/p/quake/quakeforge
synced 2025-03-19 17:01:46 +00:00
fix a glaring security hole
This commit is contained in:
parent
0dffb8bfcb
commit
db929ff2ef
1 changed files with 6 additions and 2 deletions
|
@ -224,11 +224,15 @@ Con_Condump_f (void)
|
|||
char name[MAX_OSPATH];
|
||||
|
||||
if (Cmd_Argc () != 2) {
|
||||
Con_Printf ("usage: condump <filename>");
|
||||
Con_Printf ("usage: condump <filename>\n");
|
||||
return;
|
||||
}
|
||||
|
||||
snprintf (name, sizeof (name), "%s/%s", com_gamedir, Cmd_Argv (1));
|
||||
if (strchr (Cmd_Argv (1), '/') || strchr (Cmd_Argv (1), '\\')) {
|
||||
Con_Printf ("invalid character in filename\n");
|
||||
return;
|
||||
}
|
||||
snprintf (name, sizeof (name), "%s/%s.txt", com_gamedir, Cmd_Argv (1));
|
||||
|
||||
if (!(file = Qopen (name, "wt"))) {
|
||||
Con_Printf ("could not open %s for writing: %s\n", name,
|
||||
|
|
Loading…
Reference in a new issue