Wireshark-commits: [Wireshark-commits] master-3.0 725c526: HL7: do not call proto_reg_handoff_hl7()
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: Mon, 01 Apr 2019 08:52:25 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=725c5264e1ebc16c5ca7288daaf9b24402e8be27
Submitter: "Pascal Quantin <pascal@xxxxxxxxxxxxx>"
Changed: branch: master-3.0
Repository: wireshark

Commits:

725c526 by Pascal Quantin (pascal@xxxxxxxxxxxxx):

    HL7: do not call proto_reg_handoff_hl7() when changing a preference
    
    None of the preference require this function to be called again and it
    it not designed to be called several times (systematically adds the
    dissector handle).
    
    Change-Id: I0aca361902e65668c991e26025236c7ebb0fe946
    Reviewed-on: https://code.wireshark.org/review/32666
    Reviewed-by: Pascal Quantin <pascal@xxxxxxxxxxxxx>
    Petri-Dish: Pascal Quantin <pascal@xxxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    (cherry picked from commit 2f09f6aba4c6bc31fc3330d3e25e54c6fdef8309)
    Reviewed-on: https://code.wireshark.org/review/32669
    

Actions performed:

    from  cb619d3   On Windows, don't do console stuff unless we're on a terminal.
     add  725c526   HL7: do not call proto_reg_handoff_hl7() when changing a preference


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