Wireshark-commits: [Wireshark-commits] master 717b5a2: rtmpt: Remove 'if (tree)'
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, 26 Jan 2015 20:09:28 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=717b5a2ecf15b7e7ad3483fa9d8c64336f8456a5
Submitter: Bill Meier (wmeier@xxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

717b5a2 by Bill Meier (wmeier@xxxxxxxxxxx):

    rtmpt: Remove 'if (tree)'
    
    Setting a var under 'if (tree)' and then using it later (not under 'if (tree)') to affect col_...()/expert_...() is not OK.
    
    Change-Id: I144b5fa50b4550cd14c604970f5d84dbf13f3e90
    Reviewed-on: https://code.wireshark.org/review/6799
    Reviewed-by: Bill Meier <wmeier@xxxxxxxxxxx>
    

Actions performed:

    from  c9579d0   3GPP: upgrade 2G/3G NAS dissector to v12.8.0 and NAS EPS dissector to v12.7.0
    adds  717b5a2   rtmpt: Remove 'if (tree)'


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