Wireshark-commits: [Wireshark-commits] master-2.4 81216a1: packet-ipmi-picmg.c: Add NULL check in r

Date Prev · Date Next · Thread Prev · Thread Next
Date Index · Thread Index · Other Months · All Mailing Lists
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Mon, 12 Feb 2018 16:00:33 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=81216a176b25dd8a616e11808a951e141a467009
Submitter: Michael Mann (mmann78@xxxxxxxxxxxx)
Changed: branch: master-2.4
Repository: wireshark

Commits:

81216a1 by Michael Mann (mmann78@xxxxxxxxxxxx):

    packet-ipmi-picmg.c: Add NULL check in rq17.
    
    Crafted packets may not have ipmi_header.
    
    Bug: 14409
    Change-Id: Ib6a8eceab13525c6c8dca5cef8bce3532dc50911
    Reviewed-on: https://code.wireshark.org/review/25745
    Petri-Dish: Michael Mann <mmann78@xxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    (cherry picked from commit 0bd863ae4696858b67b72bf1b0a32377877a5a38)
    Reviewed-on: https://code.wireshark.org/review/25754
    Reviewed-by: Michael Mann <mmann78@xxxxxxxxxxxx>
    

Actions performed:

    from  78af50f   WCCP: Prevent very long loop in dissect_wccp2_alternate_mask_value_set_element
    adds  81216a1   packet-ipmi-picmg.c: Add NULL check in rq17.


Summary of changes:
 epan/dissectors/packet-ipmi-picmg.c | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)