[Perl] Tainting & unlink

Pagina: 1
Acties:

  • Juup
  • Registratie: Februari 2000
  • Niet online
Als m'n scrippie tainting aan heeft staan (met de -T optie) dan wordt dir stukje code afgekeurd met de melding:

Insecure dependency in unlink while running with -T switch at /webmount/home/teijsden/cgi-bin/cms.pl line 159.

Dit zijn regel 156 en verder:
code:
1
2
3
4
5
6
7
$filename = "user inputted & tainted value"; #tainted
if ($filename =~ /^\Q$htdocsdir\E\/([a-z0-9_\-\/]+\.html)$/)
{ my $subFileName = $1;
  if (-f "$htdocsdir/$subFileName")
  { unlink ("$htdocsdir/$subFileName"); #regel 159
  }
}

Ik doe dus zoals dat Taint gebeuren voorschrijft een nette regexp op die filename. Toch die foutmelding. Wat doe ik fout?

Een wappie is iemand die gevallen is voor de (jarenlange) Russische desinformatiecampagnes.
Wantrouwen en confirmation bias doen de rest.


  • JayTaph
  • Registratie: Oktober 1999
  • Laatst online: 28-11-2025

JayTaph

Portability is for canoes.

En $htdocsdir?

Yo dawg, I heard you like posts so I posted below your post so you can post again.


  • Juup
  • Registratie: Februari 2000
  • Niet online
$htdocsdir is untainted. Echt ;)

Een wappie is iemand die gevallen is voor de (jarenlange) Russische desinformatiecampagnes.
Wantrouwen en confirmation bias doen de rest.


  • JayTaph
  • Registratie: Oktober 1999
  • Laatst online: 28-11-2025

JayTaph

Portability is for canoes.

Op donderdag 28 februari 2002 10:26 schreef Jaaap het volgende:
$htdocsdir is untainted. Echt ;)
Ok, dan geloof ik je :)

Maar ik zie zo geen oplossing in ieder geval. Op de een of andere manier is OF $htdocsdir OF $subFileName OF de combinatie van beide tainted. Een doekje voor het bloeden zou zijn om deze in een aparte var te zetten en daarna met een default regex te untainten.

Yo dawg, I heard you like posts so I posted below your post so you can post again.


  • Juup
  • Registratie: Februari 2000
  • Niet online
Ik heb het gevonden. Stom foutje. Nix interessants.

Een wappie is iemand die gevallen is voor de (jarenlange) Russische desinformatiecampagnes.
Wantrouwen en confirmation bias doen de rest.


  • JayTaph
  • Registratie: Oktober 1999
  • Laatst online: 28-11-2025

JayTaph

Portability is for canoes.

en nou wil ik het weten ook :+

Yo dawg, I heard you like posts so I posted below your post so you can post again.

Pagina: 1