Wireshark · Wireshark-dev: Re: [Wireshark-dev] [Wireshark-commits] master a37ac98: SCTP: fix dissection of DATA chunks

[Pascal Quantin <pascal.quantin@xxxxxxxxx>]

Hi Michael,

2015-10-07 8:41 GMT+02:00 Michael Tuexen <Michael.Tuexen@xxxxxxxxxxxxxxxxx>:

Hi Pascal,

thanks for finding and fixing it! Can you cherry pick it also for 2.0?

You are welcome :)

I did not cherry-pick it because I thought the I-DATA development was not part of the branch. But it looks like I was wrong (as the branch was created right after). It's now done.

Regards,

Pascal.

Best regards
Michael
> On 06 Oct 2015, at 23:31, Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx> wrote:
>
> URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=a37ac98c5eb5c44794ca36418f93e7ed1c3e0af5
> Submitter: Pascal Quantin (pascal.quantin@xxxxxxxxx)
> Changed: branch: master
> Repository: wireshark
>
> Commits:
>
> a37ac98 by Pascal Quantin (pascal.quantin@xxxxxxxxx):
>
>    SCTP: fix dissection of DATA chunks
>
>    Regression introduced in gd52322e
>
>    Change-Id: I57baf53d81c7e95ea8ad15e4799033d341e4ee61
>    Reviewed-on: https://code.wireshark.org/review/10845
>    Reviewed-by: Pascal Quantin <pascal.quantin@xxxxxxxxx>
>
>
> Actions performed:
>
>    from  12c83d0   Report an error if we don't find fop and are going to try to use it.
>    adds  a37ac98   SCTP: fix dissection of DATA chunks
>
>
> Summary of changes:
> epan/dissectors/packet-sctp.c |    2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
> ___________________________________________________________________________
> Sent via:    Wireshark-commits mailing list <wireshark-commits@xxxxxxxxxxxxx>
> Archives:    https://www.wireshark.org/lists/wireshark-commits
> Unsubscribe: https://wireshark.org/mailman/options/wireshark-commits
>             mailto:wireshark-commits-request@xxxxxxxxxxxxx?subject=unsubscribe
>