Wireshark · Wireshark-dev: Re: [Wireshark-dev] [Wireshark-commits] master fc5d8db: Create the HTTP tree after we're assured it'

[Evan Huus <eapache@xxxxxxxxx>]

The DTLS decryption test has been failing for the last few days
because of this. Not sure what the relation is, but...

On Sun, Apr 20, 2014 at 8:40 AM, Wireshark code review
<code-review-do-not-reply@xxxxxxxxxxxxx> wrote:
> URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=fc5d8db74dc473610b9fc6c0c4b571d4aa65264a
> Submitter: Michael Mann (mmann78@xxxxxxxxxxxx)
> Changed: branch: master
> Repository: wireshark
>
> Commits:
>
> fc5d8db by Michael Mann (mmann78@xxxxxxxxxxxx):
>
>     Create the HTTP tree after we're assured it's HTTP.  Otherwise a bogus tree is created when HTTP2 traffic is found.
>
>     Change-Id: Ic315ed9b7d65fe70401945cb0cceda4af863d140
>     Reviewed-on: https://code.wireshark.org/review/1215
>     Reviewed-by: Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>
>     Reviewed-by: Michael Mann <mmann78@xxxxxxxxxxxx>
>
>
> Actions performed:
>
>     from  d0489f2   Clean up white space (replace tabs with 4 spaces).
>     adds  fc5d8db   Create the HTTP tree after we're assured it's HTTP.  Otherwise a bogus tree is created when HTTP2 traffic is found.
>
>
> Summary of changes:
>  epan/dissectors/packet-http.c |   11 ++++++-----
>  1 file changed, 6 insertions(+), 5 deletions(-)
> ___________________________________________________________________________
> Sent via:    Wireshark-commits mailing list <wireshark-commits@xxxxxxxxxxxxx>
> Archives:    http://www.wireshark.org/lists/wireshark-commits
> Unsubscribe: https://wireshark.org/mailman/options/wireshark-commits
>              mailto:wireshark-commits-request@xxxxxxxxxxxxx?subject=unsubscribe