Wireshark-commits: [Wireshark-commits] master 8ad5611: sv: remove some unnecessary checks

From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Wed, 26 Aug 2015 20:57:16 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=8ad56117502207d95d54bd2c8dff24d001530d24
Submitter: Martin Kaiser (wireshark@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

8ad5611 by Martin Kaiser (wireshark@xxxxxxxxx):

    sv: remove some unnecessary checks
    
    http://personal.strath.ac.uk/robert.c.atkinson/papers/sgms2011.pdf
    implies that appid, length and the two reserved fields are mandatory
    just try to dissect them, don't check if there's enough bytes remaining
    
    correct a typo while at it
    
    Change-Id: I09a315fc2c70161af527dca5661f877e09b63ff6
    Reviewed-on: https://code.wireshark.org/review/10273
    Reviewed-by: Martin Kaiser <wireshark@xxxxxxxxx>
    

Actions performed:

    from  7b35992   Reload IO Graph Y fields when reload Lua plugins
    adds  8ad5611   sv: remove some unnecessary checks


Summary of changes:
 asn1/sv/packet-sv-template.c |   16 ++++++----------
 epan/dissectors/packet-sv.c  |   20 ++++++++------------
 2 files changed, 14 insertions(+), 22 deletions(-)