Incidencia #45501

PACKET_CONN_PING_INFO has 8bit connections count

Abrir Fecha: 2022-08-29 01:17 Última actualización: 2022-09-10 16:01

Informador:
Propietario:
Tipo:
Estado:
Cerrado
Componente:
Prioridad:
5 - Medium
Gravedad:
5 - Medium
Resolución:
Fixed
Fichero:
3

Details

Max number of connections is at least 1000 in S3_1 and master. Yet PACKET_CONN_PING_INFO has just UINT8 for storing number of connections.

Ticket History (3/7 Histories)

2022-08-29 01:17 Updated by: cazfi
  • New Ticket "PACKET_CONN_PING_INFO has 8bit connections count" created
2022-08-29 07:33 Updated by: cazfi
Comentario

UINT8 is actually too small even in S3_0.

In S3_0 this needs to be fixed with a optional capability. So does #45499, if we decide to do it for S3_0 too. For S3_0 part, it's best to handle them together (in the same commit), so that we don't need to introduce two separate capabilities but have both issues fixed under the same capability.

2022-09-04 13:47 Updated by: cazfi
  • Propietario Update from (Ninguno) to cazfi
  • Resolución Update from Ninguno to Accepted
Comentario

Keeping marked S3_1-npf as the target milestone, as that's one hard deadline. The S3_0 fix is very much meant to be pushed in regardless.

For not breaking the frozen protocol, but to have an optional capability, S3_0 patch is far more complex than the one for later branches.

2022-09-10 16:01 Updated by: cazfi
  • Estado Update from Open to Cerrado
  • Resolución Update from Accepted to Fixed

Editar

Please login to add comment to this ticket » Entrar