Wireshark-commits: [Wireshark-commits] master cbb68ba: if the original packet already contains a fi
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Tue, 5 Aug 2014 19:52:12 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=cbb68ba64de3db5b9649beb6bde095e2b8964464
Submitter: Martin Kaiser (wireshark@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

cbb68ba by Martin Kaiser (wireshark@xxxxxxxxx):

    if the original packet already contains a file name,
    don't add a generated tree item with the filename stored in the
    conversation
    
    Change-Id: Ia9a7fe712d0dd8241399e456509019b8e3902554
    Reviewed-on: https://code.wireshark.org/review/3447
    Reviewed-by: Martin Kaiser <wireshark@xxxxxxxxx>
    Tested-by: Martin Kaiser <wireshark@xxxxxxxxx>
    

Actions performed:

    from  d60ce0b   no need for if(tree) display the opcode first, followed by the file name
    adds  cbb68ba   if the original packet already contains a file name, don't add a generated tree item with the filename stored in the conversation


Summary of changes:
 epan/dissectors/packet-tftp.c |   23 +++++++++++++----------
 1 file changed, 13 insertions(+), 10 deletions(-)