Wireshark-dev: Re: [Wireshark-dev] Max size of a field seems to be 240 for a dissector

From: Gerald Combs <gerald@xxxxxxxxxxxxx>
Date: Tue, 18 Oct 2022 10:34:00 -0700
On 10/18/22 9:49 AM, Richard Sharpe wrote:
On Tue, Oct 18, 2022 at 9:39 AM Pascal Quantin <pascal@xxxxxxxxxxxxx> wrote:

Hi Richard,

Le mar. 18 oct. 2022 à 18:30, Richard Sharpe <realrichardsharpe@xxxxxxxxx> a écrit :

Hi folks,

How do I squeeze more than 240 chars into a string field?

You can't currently.  As seen in epan/proto.h, you have the ITEM_LABEL_LENGTH define set to 240 and that exists since ages.


I am trying to fix an issue with the beamforming matrices for 802.11ax
and 802.11be and maybe 802.11ac.

I guess you refer to https://gitlab.com/wireshark/wireshark/-/issues/18504

Yes.


When I try to assemble all of a single SCIDX, I get this:
----------------------------
S [truncated]: SCIDX: -122, phi11:57, phi21:34, phi31:59, phi41:8,
phi51:52, phi61:18, phi71:14, psi21:10, psi31:8, psi41:1, psi51:5,
psi61:5, psi71:3, psi81:4, phi22:33, phi32:22, phi42:22, phi52:10,
phi62:41, phi72:59, psi32:8, psi42:3,
------------------------------

Why is there an arbitrary limit? Is it because I am working with 3.4.8?

No, it has been like this for 23 years according to Git blame.
Maybe it's time to display this matrix differently (with several lines?) instead of appending all those infos in a single line?

I tried that but the user finds it hard to work with.

I have found a way to make the user happy, but of the spec changes to
increase the number of angles needed we will run into the same
problem.

Would using "Φ" in place of "phi" and "Ψ" in place of "psi" make sense? The Greek letters use two UTF-8 bytes instead of three, which would let you fit more elements into the available space.