IRC Dev/ParchesIRCDOct2003

IRC-DEV

PaginaDeEntrada
- ParchesIRCD
  - ParchesIRCDOct2003

Usuario:
Anonymous User

CambiosRecientes VerAyuda

La última edición fue hecha por jcea, el 2003-10-31 14:59:25

09/Oct/03 NiKoLaS deja IRC-Dev - 30/May/03: Los jupes/links de nodos se controlan por BDD. Asignado: NiKoLaS

09/Oct/03 NiKoLaS deja IRC-Dev - 12/Sep/03: Si alguien entra en un canal con baneos y se cambia de nick a uno "baneado", puede hablar en el canal con normalidad. Asignado: NiKoLaS

DONE 22/Oct/03 - 17/Sep/03: El script configure debe de pasarles los parametros que recibe a config/configure y zlib/configure. Asignado: Amn3S1A

01/Oct/03: Pasar la tabla o a FLAGS individuales. Osea, cada bit, un flag. Asignado: N3TKaT

DONE 01/Oct/03 - 01/Oct/03: Evitar buscar a la BDD la clave de cifrado de IPs, siempre que sea posible. Optimizar también su conversión ASCII a binario. Asignado: mount

01/Oct/03: Crear una rutina que sea invocable por "m_nick_local" y "m_rename" que se encargue de poner un nick nuevo a un usuario, ponerle un nick "invitado" y se encargue de todo el tema de ajustar modos apropiadamente. Debe ajustar correctamente también los contadores de +h, etc. Asignado: mount

DONE 21/Oct/03 - 08/Oct/03: Traducción al castellano del archivo INSTALL. Asignado: CaZoN

DONE 10/Oct/03 - 10/Oct/03: Separar "set_mode" en "set_mode_local" y "set_mode_remoto". Asignado: jcea

10/Oct/03: La parte final de "set_mode_local" y "set_mode_remoto", que es la que activa y propaga modos, es común para ambas rutinas y puede sacarse "factor común". Asignado: jcea

10/Oct/03: En "set_mode_*" hay muchas confusiones en el código original entre "sptr" y "cptr". Es un milagro que no haya fallado nunca. Asignado: jcea

10/Oct/03: En "set_mode_remoto" se puede limpiar código de "IsOperOnLocalChannel" y "OPER_MODE_LCHAN". Asignado: jcea

21/Oct/03 Con el parche u2.10.H.08.52, esto está anticuado - 13/Oct/03: Si no se especifica ninguna clave para la autentificación del nick, no ha de penalizarse el target. Asignado: zoltan
- http://mailman.argo.es/pipermail/irc-dev/2003-October/003315.html

DONE 15/Oct/03 - 15/Oct/03: Cambiar todos los usos del tabulador por espacios, seguido de un "make indent". Asignado: jcea

DONE 20/Oct/03 - 20/Oct/03: Error en el db_pack. Asignado: RyDeN

DONE 21/Oct/03 - 21/Oct/03: Mover el limpiado de la caché de las ips virtuales a "s_bdd.c". Asignado: RyDeN

DONE 21/Oct/03 - 21/Oct/03: Revocar los parches u2.10.H.07.19, u2.10.H.07.21 y u2.10.H.07.62. Asignado: jcea

DONE 21/Oct/03 - 21/Oct/03: Si se modifica un valor en la tabla "v" o "w", hacer que tenga un efecto instantáneo. Asignado: RyDeN

DONE 21/Oct/03 - 21/Oct:03: Regenerar las ips virtuales solo cuando sea necesario. Asignado: RyDeN

DONE 22/Oct/03 - 21/Oct/03: Solucionar el problema con las optimizaciones de "TieneIpVirtualPersonalizada()", cuando se cambia de nick entre nicks con IP personalizada o no, y las distintas configuraciones. Asignado: RyDeN

DONE 22/Oct/03 - 22/Oct/03: Responder con ERR_CANNOTSENDTOCHAN en el caso de que el usuario no pueda enviar tanto a canales como a privados. Asignado: Daijo

DONE 29/Oct/03 - 22/Oct/03: Cuando se propaga un nuevo registro "v" para alguien, le llega dos veces el "numeric" de "XXX is now your hidden host". El primero con la IP cifrada y el segundo con la IP personalizada. La forma más sencilla de solucionarlo es que si la notificación de borrado es por un reemplazo, no recalcular la virtual aún, a la espera de la inminente llamada para una nueva alta. Asignado: RyDeN

DONE 30/Oct/03 - 29/Oct/03: Los forbids y suspends por BDD deben tener efecto inmediato. Asignado: mount

29/Oct/03: Cuando se propaga por la BDD el registro que activa la inserción o la eliminación de la IP cifrada para los usuarios en la tabla "w", debe hacerse limpieza de la caché de IPs virtuales. Asignado: RyDeN

DONE 29/Oct/03 - 29/Oct/03: En el parche "u2.10.H.08.57", se pueden sacar cosas "factor común" a una rutina independiente. Asignado: RyDeN

DONE 31/Oct/03 - 29/Oct/03: Aunque un canal no tenga clave, se reserva espacio en memoria para ella. Creo que 24 bytes, o algo así. Asignado: RyDeN

Parches pendientes de asignar

Cuando toda la red esté actualizada (u2.10.H.08.55), eliminar el registro "permite.nicks.largos.en.local".

Cuando un nodo hace "bounce" de cambios de modos de un usuario, debe mandarle también un "-o" para quitar el op a dicho usuario y "curar" el desync.

Reescribir el código para evitar los warning "dereferencing type-punned pointer will break strict-aliasing rules".

Este problema parece complicado de reproducir:

 [17:13] <[_-NeO-_]> chkconf.c:67: redefinition of `struct Client'
 [17:13] <[_-NeO-_]> make[1]: *** [chkconf.o] Error 1

"Slab allocator" para cadenas cortas.

Cuando se carga la BDD de disco duro, al arrancar el IRCD, no hay ningún usuario conectado. Aprovechar ese hecho para optimizar el tema de regenerar las IPs virtuales, nicks prohibidos/suspendidos, etc.

DONE 31/Oct/03 - Aparentemente se puede llamar a "m_nick_local" con un usuario que NO es un usuario (¿aún?), fallando el "assert()" en "make_virtualhost()":

#0  0xef5fb55c in _libc_kill () from /usr/lib/libc.so.1
#1  0xef5bfbe8 in abort () from /usr/lib/libc.so.1
#2  0xef5bfc48 in _assert () from /usr/lib/libc.so.1
#3  0x000432c4 in make_virtualhost (acptr=, mostrar=1, simular_registrado=0) at s_user.c:3153
#4  0x000443d8 in m_nick_local (cptr=, sptr=, parc=1179648, parv=0x9bdd0) at s_user.c:4539
#5  0x0002c56c in parse_client (cptr=0x12dd70, buffer=0x60000 "\220`?ÿ\200\220@\b2\200", bufend=0x9bdd0 "") at parse.c:502
#6  0x0002be88 in client_dopacket (cptr=0x12dd70, length=14) at packet.c:244
#7  0x00036310 in read_packet (cptr=, rfd=653008) at s_bsd.c:1764
#8  0x0003405c in read_message (delay=4) at s_bsd.c:2126
#9  0x00022670 in main (argc=535552, argv=0x82c00) at ircd.c:929

Para solucionarlo cambio

  if (IsHidden(sptr))
    make_virtualhost(sptr, 1, 0);

por

  if (IsHidden(sptr) && IsUser(sptr))
    make_virtualhost(sptr, 1, 0);

en la línea 4539 de "ircd/s_user.c", pero el cambio no me parece satisfactorio. Analizar, diagnosticar y corregir.

31/Oct/03 Aclarado en u2.10.H.08.74 y u2.10.H.08.75.

Pensar qué debe pasar con la caché de IPs virtuales cuando se cambia la clave de cifrado de IPs de la red. ¿Se borra toda la caché?. ¿Se comunica las nuevas virtuales a los usuarios?.

Cuando hay varias lineas C/N en varias clases, con tiempos de conexión diferentes para simular prioridades, si se conecta una linea de alta prioridad y la conexión se corta (por ejemplo, clave incorrecta), pues persiste en reconectar con esa conexión de alta prioridad, en vez de probar otras.

Si hay varios %LOCALMOTD en el MOTD distribuido por BDD, sólo se muestra el primero. Solucionarlo o documentarlo.

Documentar o eliminar el uso de la variable "count" en la rutina de MOTD.

No se ve lo mismo si no hay MOTD local que si no lo hay pero existe una única linea "%LOCALMOTD" en el MOTD de la BDD.

En el MOTD aparece la fecha y la hora. Poner también el "timezone".

Los admins no deberían tener acceso a la BDD de nicks, aunque las claves estén "hashed". No lo necesitan.

"Numerics" generados por servidor. Máscara larga, para que no se repitan "numerics" aunque haya mucho lag en la red.

Aclarar y documentar el uso de "simular_registrado" en "make_virtualhost()".

En el "make config", el texto del "watch" tiene al menos una errata.

Se declara en varios sitios "static void send_umode_out(aClient a, aClient b, int c, int d, int e);". Hacer limpieza.

Las cadenas con tamaño estático en las estructuras de clientes, canales, etc., que no suelen aprovechar su tamaño (por ejemplo, se define un tamaño de X bytes pero se suele usar mucho menos; ejemplos: clave de canal, topic, nombre de canal, hostname del usuario, etc), pasarlas a malloc/free, con vista a migrarlas de nuevo al futuro "slab allocator".