Wireshark · Wireshark-dev: Re: [Wireshark-dev] Familiar with gtk/sctp_graph_dlg.c?

[Chris Maynard <chris.maynard@xxxxxxxxx>]

Stephen Fisher <steve@...> writes:

> Anyone familiar with gtk/sctp_graph_dlg.c?  This line in particular 
> (#270):
> 
> dup_list = (guint32 *)(((char
*)&sack_header->nr_of_dups)+sizeof(guint16)+(nr*sizeof(struct 
> gaps)));
> 
> I'm trying to investigate a clang error about this increasing required 
> alignment from 1 to 4 and the complexity of that line is making my brain 
> hurt :)

Ouch!  I haven't spent a ton of time looking at this, but I don't understand it
either.  Maybe the authors - Irene and/or Varun - could be of help?

By the way, while looking around a bit, I think I found a memory leak at line 
1555:

1555:	u_data->assoc=g_malloc(sizeof(sctp_assoc_info_t));
1556:	u_data->assoc=userdata->assoc;

I assume line 1555 should be deleted?