Wireshark-commits: [Wireshark-commits] master-1.12 be9397b: Fix a couple of bugs in IPMI dissector:
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: Thu, 26 Feb 2015 08:54:45 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=be9397bc9e7527d483f6342e13ab8f9c3647c0ea
Submitter: Alexis La Goutte (alexis.lagoutte@xxxxxxxxx)
Changed: branch: master-1.12
Repository: wireshark

Commits:

be9397b by Dmitry Bazhenov (dima_b@xxxxxxxxxxxxxxx):

    Fix a couple of bugs in IPMI dissector:
    
    * Fix incorrect offset for Block Selector field
      in Get LAN Configuration Parameter request.
    * Expect 0x83 completion code ("Attempt to read write-only parameter")
      for Get LAN Configuration Parameter command instead of
      Set LAN Configuration Parameter command.
    
    Change-Id: I5be37c5f96db91b70329bfce906ce76f7b8b6958
    Reviewed-on: https://code.wireshark.org/review/7403
    Reviewed-by: Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>
    (cherry picked from commit b034870a4d318f5071ccd443f274fd610ee685f5)
    Reviewed-on: https://code.wireshark.org/review/7405
    

Actions performed:

    from  acbd5ac   Make the V10 path a bit more like the other paths.
    adds  be9397b   Fix a couple of bugs in IPMI dissector:


Summary of changes:
 epan/dissectors/packet-ipmi-transport.c |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)