Wireshark-commits: [Wireshark-commits] master-2.2 a6f89d6: echo: decide on req/resp based on matche
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Sat, 3 Dec 2016 12:46:42 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=a6f89d6fc21ffe247a7d0d4c1210a1b468067e39
Submitter: Michael Mann (mmann78@xxxxxxxxxxxx)
Changed: branch: master-2.2
Repository: wireshark

Commits:

a6f89d6 by Jaap Keuter (jaap.keuter@xxxxxxxxx):

    echo: decide on req/resp based on matched port
    
    Since the use of 'Decode as' can result in another port number than the
    default port (7) being the service port, this should be checked against.
    
    Change-Id: I93383613115595fff621e2fb9ab7959cd448c01e
    Reviewed-on: https://code.wireshark.org/review/18991
    Reviewed-by: Jaap Keuter <jaap.keuter@xxxxxxxxx>
    Petri-Dish: Jaap Keuter <jaap.keuter@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Peter Wu <peter@xxxxxxxxxxxxx>
    (cherry picked from commit 54f48c8b92475523a49c15cd549fb5408e96ea8d)
    Reviewed-on: https://code.wireshark.org/review/19048
    Reviewed-by: Michael Mann <mmann78@xxxxxxxxxxxx>
    

Actions performed:

    from  043e92a   packet-rpcrdma: Fix protocol frame pre-detection
    adds  a6f89d6   echo: decide on req/resp based on matched port


Summary of changes:
 epan/dissectors/packet-echo.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)