[Pregunta]: ¿Acarrea alguna vulnerabilidad?

[@XSStringManolo]

Te estás complicando la vida mucho. Y lo que comentas ahora no tiene relación con la pregunta inicial.

Agrega a la tabla los permisos que tiene el usuario. Por ejemplo número del 1 al 5.

1 usuario baneado.
2 visitantes.
3 usuario loggeado común.
4 moderadores, administradores.
5 webmaster (tú).

En base a esto cuando se solicite una acción compruebas si el usuario que la solicita tiene permisos para realizar la acción.

[AlbertoBSD]

Te estás complicando la vida mucho. Y lo que comentas ahora no tiene relación con la pregunta inicial.

TE ESTAS COMPLICANDO LA VIDA
X1000

Hace rato le esta por comentar lo mismo, mas una respuesta extensa, pero se me cerro la pagina y mejor asi lo deje.

[Leguim]

Te estás complicando la vida mucho. Y lo que comentas ahora no tiene relación con la pregunta inicial.

Agrega a la tabla los permisos que tiene el usuario. Por ejemplo número del 1 al 5.

1 usuario baneado.
2 visitantes.
3 usuario loggeado común.
4 moderadores, administradores.
5 webmaster (tú).

En base a esto cuando se solicite una acción compruebas si el usuario que la solicita tiene permisos para realizar la acción.

emm nose experimenta lo que te dije y te vas a dar cuenta del hueco de "vulnerabilidad" o bugeaso que puede haber con lo que dije... con lo de los permisos... nose ya que tengo una tabla aparte donde se almacenan todos los usuarios baneados, donde va a guardar razón de baneo, la id del admin que lo bloqueo, la fecha, etcétera... después permiso visitantes? por ahí para lo que estoy trabajando no me sirva de mucho porque no es un blog, después 4. moderadores, adeministradores y 5. webmaster eso lo tengo en la tabla pero se llama rango... donde 1. va a ser ayudante, 2. X, y 3. Z

[AlbertoBSD]

Si te comentan las cosas es por experiencia.

Espero que programes muy bien y aprendas lo necesario de seguridad para evitar ser hackeado, como vas hasta ahora, tarde o temprano te van a hackear y por la forma que te complicas la vida con ese tipo de códigos los fix van a ser algo tedioso llegando a tener que cambiar el paradigma de programación que usas actualmente.

Saludos!

[MinusFour]

Una banda de respuestas  pero no hacia falta todo eso, yo decía si acarreaba una vulnerabilidad no volver a definir x variable en este caso $_GET

Respondiendo: La aplicación web está en desarrollo por eso no tengo una URL que mandarles para que la vean.

A lo que iba era digamos que yo tengo un comentario que quiero editar, yo me ilustro o imagino que hay como "2 procesos" el "protocolo de seguridad entrante" y el "protocolo de seguridad de envío" (esto me lo invente yo  por si les suena raro)

Bueno el protocolo de seguridad entrante se va a encargar de validar si el usuario tiene permisos para ingresar a x pagina.php

Por ejemplo si el usuario tiene la session iniciada, si el usuario no está bloqueado, si el usuario tiene el email verificado, entre tantas otras que pueda tener una aplicación... dependiendo el caso va a redireccionarlo a otras paginas... por ejemplo si no tiene la session iniciada lo manda a el index pero si si tiene la session iniciada pero esta baneado lo mando a ban.php y así hasta completar todos los protocolos.

Después está el protocolo de seguridad de envío que es cuando un usuario envía un formulario... Generalmente van a compartir el mismo protocolo pero con una diferencia y es que el protocolo de envío va a preguntar si la $session_id es igual a $_SESSION['id_user']

hay les explico (me va a salir tremendo texto bíblico pero así se entiende xD)

Ejemplo:

Código (php) [Seleccionar] Expandir


// protocolo de seguridad entrante
[Si hay sessión iniciada]
     -> [Si usuario no esta bloqueado]
           -> [Se mantiene a el usuario en la pagina]
                [Creo una variable $session_id que va a tener el valor de $_SESSION['id_user']
     [Si no: mando -> ban.php]
[Si no: mando-> index.php]


Imaginen que una persona tiene abierto una pagina comentar.php que requiere que el usuario tenga una session inciada pero ese mismo usuario abre otra pagina comentar.php, hay ahora 2 pestañas abiertas con el mismo fichero... ahora en la primer pestaña va a cerrar su sesión por lo que lo va mandar a el index pero recordemos que dejó abierta otra donde se ve la pagina como si tuviera la sesión todavía iniciada, gracias a el protocolo de envío lo que va a hacer es que cuando el usuario que tiene una pagina abierta que requiere tener la sessión iniciada pero no la tiene lo que va a hacer es redirigirlo cuando intente usar el formulario...

Ahora gracias a session_id lo que va a hacer es que imaginemos ahora que el usuario tiene 2 cuentas, y abre esas 2 pestañas con el mismo fichero php cierra la sesión en una, pero le quedo la otra abierta y vuelve a ingresar a otra cuenta diferente pero ahora tenemos (1). comentar.php con una cuenta (X) y tenemos comentar.php con una cuenta (Y) ...

el protocolo de envio se va a quedar corto porque si va  a detectar que hay una session abierta, no puede decir si es la misma de antes pero si que hay una... entonces con session id lo que hago es compararlas

$session_id == $_session['id_user'] ?

Lo que trato de mostrar con esto (igual ya deben saber) es que cuando el usuario use el formulario van haber datos que se van a "actualizar" que no van a seguir siendo iguales...

Código (php) [Seleccionar] Expandir


<?php
$comentarios = comentarios::Mostrar_Comentarios(); // Muestra todos los comentarios
$comentarios = count($comentarios); // contamos todos los registros

echo $comentarios; // hay 3 comentarios
?>

Ahora imaginemos cuando el usuario manda un formulario

<?php
if(isset($_POST['ver_otravez']))
{
     $comentarios = comentarios::Mostrar_Comentarios(); // Muestra todos los comentarios
     $comentarios = count($comentarios); // contamos todos los registros

     echo $comentarios; // capaz acá pueda mostrar 4 o 5 o capaz menos comentarios, 2 comentarios... porque desde que yo abro la pagina hasta que yo mando un formulario pueden mostrarse datos diferentes (actualizados)

     Si yo acá no volvía a definir la variable $comentarios se iba a quedar con un valor viejo que era el 3
}
?>


A eso iba cuando preguntaba si tenia que volver  a definir variables cuando envío el formulario... por ahí un $_GET no se puede actualizar de otra forma que no sea cambiandolo y recargando la pagina por eso preguntaba...
Yo se que es un poco dificil de entender, más que nada no porque sea dificil sino porque explico para la mrd xD

En ningún momento el usuario vas a tener dos cuentas abiertas por navegador. La única forma de hacer tal cosa sería quizás usar diferentes perfiles de navegador (me imagino que no se comparte la información de navegador en esta instancia, pero nunca lo he probado), usar diferentes navegadores o usar alguna especie de script del lado del cliente que maneje las session ids.

Si tienes un usuario en comentar.php con una sesión iniciada y abre una nueva pestaña, no estás creando dos sesiones en ningún momento. Y si el usuario aborta la sesión en una pestaña, esa sesión ya no existe del lado del servidor (o al menos esa es la definición de borrar una sesión). No se crea una nueva sesión mágicamente.

Las sesiones no se crean del lado del cliente. Tendrías TU que crear un sistema que maneje multiples sesiones (a lo cual no le encuentro un uso práctico, pero quizás lo hay).

No entiendo para que quieres una variable $session_id, ¿Estás abriendo sesiones sin que el usuario haya iniciado sesión? Viene siendo lo mismo, si usas una variable dentro de la sesión para determinar si ha iniciado sesión, al momento que el usuario aborte la sesión tu borras esa variable del lado del servidor. Ya no existe.