Thus wrote Martin Kaiser ([email protected]): > Thus wrote Jakub Zawadzki ([email protected]):
> > proto_item_fill_label() allows empty bytes: > > 5389 case FT_BYTES: > > 5390 case FT_UINT_BYTES: > > 5391 bytes = (guint8 *)fvalue_get(&fi->value); > > 5392 label_fill(label_str, 0, hfinfo, > > 5393 (bytes) ? bytes_to_str(bytes, > > fvalue_length(&fi->value)) : "<MISSING>"); > > 5394 break; > > It'd be good to make them consistent, allow empty bytes (+1 from me) or > > assert in both place. > ok, I'll change this. done in r54290. ___________________________________________________________________________ Sent via: Wireshark-dev mailing list <[email protected]> Archives: http://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev mailto:[email protected]?subject=unsubscribe
