Wireshark-commits: [Wireshark-commits] master-2.2 b19a088: openSAFETY: Add two checks for scm udid
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: Wed, 3 Aug 2016 11:18:40 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=b19a08874915bbf303a3b6d4b4df9b6e3799517b
Submitter: Anders Broman (a.broman58@xxxxxxxxx)
Changed: branch: master-2.2
Repository: wireshark

Commits:

b19a088 by Roland Knall (roland.knall@xxxxxxxxxxxxxxxxx):

    openSAFETY: Add two checks for scm udid validity
    
    This is being done, to prevent false-positives which can
    not be filtered out using the heuristics, but have to be
    caught using additional check measurements
    
    Change-Id: I2ff2c97decf8a93d43f8f5b54e4d147552970b3f
    Reviewed-on: https://code.wireshark.org/review/16843
    Petri-Dish: Roland Knall <rknall@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Roland Knall <rknall@xxxxxxxxx>
    (cherry picked from commit e631e1156daa25671f4b61b92f9bee646866d0c2)
    Reviewed-on: https://code.wireshark.org/review/16853
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  50b65f1   Set correct length for Req_Seq_Nr field
    adds  b19a088   openSAFETY: Add two checks for scm udid validity


Summary of changes:
 epan/dissectors/packet-opensafety.c |  106 ++++++++++++++++++++++++-----------
 1 file changed, 73 insertions(+), 33 deletions(-)