PHP - Validacion - No encuentro el error

echo "<h1>Tabla de fotos con enlace</h1>";

function valida_fotos($fotos){

    $rdo=0;

    if(preg_match("/[Jj][Pp][Gg]$/",$fotos))rdo=1;

    if(preg_match("/[Gg][Ii][Ff]$/",$fotos))rdo=1;

    if(preg_match("/[Pp][Nn][Gg]$/",$fotos))rdo=1;

    if(preg_match("/[Bb][Mm][Pp]$/",$fotos))rdo=1;

return $rdo;

}

echo "<table border=1>";

$puntero=opendir('fotos');

$i=1;

while (false!==($foto=readdir($puntero))){

    if($foto!="."&&$foto!=".."&&valida_envia($foto)){

        if($i==1)

        echo "<tr>";

        echo "<td><a href='fotos/$foto'>";

        echo "<img src='fotos/$foto'width=100 height=100</img>";

        echo "</a></td>";

        if($i==4){

            echo "<tr>";$i=0;}

            $i++;

    }

}

closedir($puntero);

echo "</table>";

<?

echo "<h1>Tabla de fotos con enlace</h1>\n\n";

/* Tu funcion era muy larga. Ademas ni la anterior ni esta

 * comprueban si es un archivo o un directorio con nombre

 * terminado en .gif,jpg... deberias usar is_file();

 *

 * El REGEX que hay puesto no distingue entre mayusculas y minusculas

 */

function valida_fotos($fotos)

{

	$rdo=preg_match("/\.(?:jpg|jpeg|gif|png|bmp)$/i",$fotos);

	return $rdo;

}

echo "<table border=1>\n";

$dir= 'fotos';

$fotos = array_diff(scandir($dir), array('..', '.'));

$i=0;

foreach($fotos as $foto)

{

	if(valida_envia($foto))//Supongo que quieres decir valida_fotos().

	{

		if($i%4==0)

			echo "\t<tr>\n";

		echo "\t\t<td>\n".

			 "\t\t\t<a href='fotos/$foto'>\n".

			 "\t\t\t\t<img src='fotos/$foto'width=100 height=100</img>\n".

			 "\t\t\t</a>\n".

			 "\t\t</td>\n";

	    if($i%4==3)

	    	echo "\t<tr>\n";//Supongo que querras decir </tr>

	    $i++;

	}

}

//Este if sirve para poner el ultimo </tr> si no se ha puesto anteriormente.

if($i%4!=3)

	echo "\t</tr>\n";

echo "</table>\n\n";