qué hay de malo en este codigo?

[aniMAYtions]

Gracias sbsyoel.
Lo malo es lo de siempre, que se trata de una intranet y el host es interno.
Voy a poneros el código completo

Código HTML:

<div id='contenedor' align="center">
        <div id='cabecera'>
            <div id='img_cabecera'>
                <img src="images/encabezado_cnse.jpg" width="90%" height="8em"/>
            </div>
            <div class='cleared'></div>
        </div>
        <div id='contenido'>
            <div id='menu_izqd'>
                <?
                    //cargaBotonesIzq();
                ?>
                <div id='img_menu_izqd' align="right">
                     <img src="images/fondo_menu_izqd.png" height="96%" />
                     <div class='cleared'></div>
                </div>
            </div>
            <div id='zona_derecha' >
                <div id='cabecera_derecha'>
                    <div id='contenido_cabecera'>
                        <div id='imagen_cabecera'></div>
                        <div id='titulo'></div>
                    </div>
                    <div id='img_cabecera_derecha' align="center">
                         <img src="images/cabecera_derecha.png" width="100%" align="left" />
                    </div>
                    
                </div>
                <div class='cleared'></div>
                <div id='contenido_derecha' style='border:1px solid green; z-index:2'>
                    <?
                        cargaLogin();
                    ?>
                </div>
                
            </div>
           <div id='pie'>
               <div id='img_pie' align="right">
                <img src="images/pie.gif" width="85%" height="5em" />
                </div>
                <div class='cleared'></div>
             </div>
            <div id='img_contenido' align="right">
                <img src="images/fondo_contenido.gif" width="85%" height="20em" />
            </div>
             <div class='cleared'></div>
             
        </div>
        
</div>
 

cargaLogin contiene el siguiente código

Código PHP:

echo"<div id='login' style='z-index=2; margin-top:3em'>
        <fieldset>
    <form name='fidentifica' method = 'post' action = '#'>
        <table>
            <tr>
                <td><input type='text' name='user' id='user' class='username' value='Nombre de usuario' onClick='javascript:vaciaInput();'/></td>
                <td class='anchotd'>&nbsp;</td>
            </tr>
            <tr>
                <td><input type='password' name='pss' class='contrasenia' value=''/></td>
                <td class='anchotd'><div class='log'><input type='button' name='bslogin' value='' class='botonlogin' src='./images/logueo/login.png' title='Enviar' onclick='xajax_login(document.fidentifica.user.value, document.fidentifica.pss.value)'/></div></td>                 
            </tr>
            <tr>
                <td colspan = '2'>
                    <span class='login'>
                        <a href='#'><span class='lostpassword' title='No recuerdas tu clave?'></span></a><a href='#'><span class='lostusername' title='No recuerdas tu usuario?'></span></a><a href='#'><span class='registration' title='Crear una cuenta nueva'></span></a>
                    </span>
                </td>
            </tr>
        </table>
        </form>
        </fieldset>
    </div>";
 

Sólo decir que este proyecto acabo de empezarlo y llevo muy poquito hecho y tiene poco más que el armazón en HTML.

Gracias de nuevo y saludos.