Wireshark · Wireshark-commits: [Wireshark-commits] master 83516dc: SMB2: fix problems with dissection of error response

[Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>]

URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=83516dce1aced2841dfd64b0311889c901296c67
Submitter: Anders Broman (a.broman58@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

83516dc by DiablosOffens (DiablosOffens@xxxxxx):

    SMB2: fix problems with dissection of error response
    
    Sometimes it isn't actually an error response but a normal packet with a non-zero status code to indicate a warning or information. This should be handled as a normal case and not break the dissection
    
    Change-Id: I7104608d67cbc7528994bd86812ea5241f1e4460
    Signed-off-by: Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>
    Reviewed-on: https://code.wireshark.org/review/12282
    Petri-Dish: Michael Mann <mmann78@xxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  78d664c   main_window: fix no previous prototype for function 'plugin_if_mainwindow_gotoframe' [-Wmissing-prototypes]
    adds  83516dc   SMB2: fix problems with dissection of error response


Summary of changes:
 epan/dissectors/packet-smb2.c |  237 +++++++++++++++++++++++------------------
 1 file changed, 132 insertions(+), 105 deletions(-)